Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add the Cheyenne creature #25

Merged
merged 1 commit into from
Feb 25, 2025

Conversation

Fabrice-TIERCELIN
Copy link

Hi @misiokles,

I'm starting to migrate the New Land of Towns creatures with this Cheyenne creature. The visual assets comes from the project and I have created the sounds. Successfully tested.

AI: Synthio Stable Audio Generator

Prompts:

  1. "magical sound, resonance, ringing"
  2. "A quick airstream"
  3. "A strong man walking on the ground"
  4. "Brief shouts of a strong man fighting"
  5. "Crackling of a campfire"

PS: Tell me if this creature was already available in VCMI

@Olegmods
Copy link
Contributor

@Fabrice-TIERCELIN why did you remove .png files from flugel submod?

@Fabrice-TIERCELIN
Copy link
Author

It's a cleanup! They are only used to build the final files.

@Olegmods
Copy link
Contributor

Olegmods commented Feb 25, 2025

It's a cleanup! They are only used to build the final files.

Well, judging by json files, they're actually used, so I guess that "cleanup" will break the mod.

Copy link

@dydzio0614 dydzio0614 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Restore flugel graphics

@misiokles
Copy link
Contributor

I've checked. ClstLibFlag01, UNFlgl1 and USFlgl1 can be deleted as not-used.

@Olegmods
Copy link
Contributor

I've checked. ClstLibFlag01, UNFlgl1 and USFlgl1 can be deleted as not-used.

These weren't deleted in this PR. Only the png graphics for the creature are deleted there, which are used by Cflugel.json.
Also I guess it's better to keep specialty icons (UNFlgl1 and USFlgl1) in case someone wants to make a hero with Flugel specialty.

@misiokles
Copy link
Contributor

Right!

@Fabrice-TIERCELIN
Copy link
Author

OK, I leave it as it is.

@misiokles misiokles merged commit 939d45b into vcmi-mods:vcmi-1.6 Feb 25, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants