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

Can specify delay in emit_fields for monsters #37895

Merged
merged 16 commits into from
Mar 2, 2020

Conversation

Fris0uman
Copy link
Contributor

@Fris0uman Fris0uman commented Feb 10, 2020

Summary

SUMMARY: Infrastructure "Can specify delay in emit_fields for monsters"

Purpose of change

Can specify delay in emit_fields for monsters
[ My Sweet Cataclysm ] Gum spiders no longer spam gum webs every turn

Describe the solution

Change emit_fileds from a set to a map of emit_id and time_durations
Fix delay for gum spider
Update json for every monster in vanilla and mods

Describe alternatives you've considered

Testing

Spawn a gum spider
No spamming webs
Wait 30 min
Spider emits one web

Spawn incandesant husk
Constant field emission

Additional context

@ZhilkinSerg ZhilkinSerg added [C++] Changes (can be) made in C++. Previously named `Code` [JSON] Changes (can be) made in JSON Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. Monsters Monsters both friendly and unfriendly. labels Feb 10, 2020
Copy link
Member

@kevingranade kevingranade left a comment

Choose a reason for hiding this comment

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

I'm not a fan of the array with the two different values, and even less a fan of putting a parser for it in units, this isn't really a standard construct we should be encouraging.
Also, you don't have to make this backwards incompatible, tit's fine to have the old style entry indicate that the duration is 1 second.

@kevingranade kevingranade merged commit 693f826 into CleverRaven:master Mar 2, 2020
@Fris0uman Fris0uman deleted the emit_delay branch March 2, 2020 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C++] Changes (can be) made in C++. Previously named `Code` Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. [JSON] Changes (can be) made in JSON Monsters Monsters both friendly and unfriendly.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants