-
Notifications
You must be signed in to change notification settings - Fork 146
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
Convert Harpy Singing Into a Trait #714
Convert Harpy Singing Into a Trait #714
Conversation
remove it from the trait menu pending discussion; if this is to make it into a generic trait; it would need to be limited in instrument choice entirely. Generic Trait = Only whatever instrument the Microphone is |
why so? |
i updated the comment, should make more sense |
It only gives microphone and some second instrument I forgot |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Making the component generic is good; I have no idea how MIDI works but there should be some way to assign instruments per component. Ideally, the singer system is just the microphone instrument and then you can reuse it and add more instruments for Harpy
Co-authored-by: Danger Revolution! <[email protected]> Signed-off-by: Mnemotechnican <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cost higher, remove species requirement
Meh Co-authored-by: VMSolidus <[email protected]> Signed-off-by: Mnemotechnican <[email protected]>
Fuck, uh... Turns out prototypedata class is apart of robust toolbox, I edited it locally, but the change wasn't committed... which is now a test fail. I need to find a way to fix that... |
This works fine now. Singing types are now defined separately from the trait and can be re-used later, which makes things like singing parrots as easy to implement as adding Also, the cost has been reduced to 1. |
test fail valid |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Signed-off-by: Mnemotechnican <[email protected]>
test fail still valid |
I'll look into it, but later |
It was a single-line fix |
# Description Adds a new trait that lets characters play midi without needing an instrument. Singer a 1-point positive trait. For now it's only available for vulps and felinids as they are naturally more vocal than other species. Also refactored harpy singing system a bit. # TODO - [X] Find out whatever causes this. Not critical, but probably should be fixed... <details><p> ![image](https://github.com/user-attachments/assets/8e0a6bf5-5fa3-4a0a-8937-2931dfe4ee62) </p></details> <details><summary><h1>Media</h1></summary> <p> Outdated ![image](https://github.com/user-attachments/assets/018b6cf3-d510-46e2-8c38-4bd49e1929bd) ![image](https://github.com/user-attachments/assets/f27bd451-b3c2-4a22-ab50-35c2f4ecbcb3) </p> </details> # Changelog :cl: - add: Added a new positive trait - singer. It allows you to play midi songs without an instrument, similarly to harpies. --------- Signed-off-by: Mnemotechnican <[email protected]> Co-authored-by: Danger Revolution! <[email protected]> Co-authored-by: VMSolidus <[email protected]>
# Description Adds a new trait that lets characters play midi without needing an instrument. Singer a 1-point positive trait. For now it's only available for vulps and felinids as they are naturally more vocal than other species. Also refactored harpy singing system a bit. # TODO - [X] Find out whatever causes this. Not critical, but probably should be fixed... <details><p> ![image](https://github.com/user-attachments/assets/8e0a6bf5-5fa3-4a0a-8937-2931dfe4ee62) </p></details> <details><summary><h1>Media</h1></summary> <p> Outdated ![image](https://github.com/user-attachments/assets/018b6cf3-d510-46e2-8c38-4bd49e1929bd) ![image](https://github.com/user-attachments/assets/f27bd451-b3c2-4a22-ab50-35c2f4ecbcb3) </p> </details> # Changelog :cl: - add: Added a new positive trait - singer. It allows you to play midi songs without an instrument, similarly to harpies. --------- Signed-off-by: Mnemotechnican <[email protected]> Co-authored-by: Danger Revolution! <[email protected]> Co-authored-by: VMSolidus <[email protected]>
Description
Adds a new trait that lets characters play midi without needing an instrument.
Singer a 1-point positive trait. For now it's only available for vulps and felinids as they are naturally more vocal than other species.
Also refactored harpy singing system a bit.
TODO
Media
Outdated
Changelog
🆑