Skip to content

More features #72

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

Merged
merged 4 commits into from
Oct 26, 2024
Merged

More features #72

merged 4 commits into from
Oct 26, 2024

Conversation

jramstedt
Copy link
Contributor

Added special handler for screens and objects with picture source. bitmap_from_tpoly_data
In multi_anim_callback it was planned to have other switching types than "random". I added selection code for it, but commented it out. I named the value "infrequency" since larger values means it switches less frequent. 2 or 3 are good values. 4 takes quite a long time to switch. If I remember correctly.
Texture preview is not perfect, but couldn't figure out a better way to show it.

Changed music index to following:
Added Music zones. Map data it is only 3 bits
Added Peril flag. It is music modifier similar to deconstructed, but is more dynamic. I think nearby enemies have something to do with it. In cyberspace it adds NUM_NODE_THEMES (2) to music zone.

ButtonControlPedestal has two targets
Added missing "scienceSecurityEquipment"

@jramstedt
Copy link
Contributor Author

jramstedt commented Aug 14, 2024

Note that displayControlPedestal picture data oveflows to target object id:s.
When using these objects user must be careful that those overflowed target object id:s are not triggerable (if that is not what user wants).
Also when setting target object ids they will modify the animation data.
This is a bug in System Shock.

@dertseha dertseha merged commit 60d7185 into inkyblackness:main Oct 26, 2024
@dertseha
Copy link
Member

Thank you for your contribution, and your patience.

I wouldn't have hardcoded in the names of the music, as some mod also that - yet I'm also the one taking months to accept a pull request :)

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.

2 participants