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

Evolution should default to 1 #77157

Open
Procyonae opened this issue Oct 19, 2024 · 3 comments
Open

Evolution should default to 1 #77157

Procyonae opened this issue Oct 19, 2024 · 3 comments
Labels
<Suggestion / Discussion> Talk it out before implementing

Comments

@Procyonae
Copy link
Contributor

Describe the bug

Evolution scaling defaults to 4 and is used in 3 places currently:
Regular evolution which is annoying but ok as it's obvious that's going to be affected so you can expect peeps to /4 the base half life
Starts/ends for monster groups which while it makes sense to be affected isn't at all intuitive that you should be /4 the days meaning the majority if not all usages are probably 4* larger than intended
This alternate method of changing monster group spawns over time that is completely unused in repo as far as I can see and is superceded by starts/ends as far as I can tell https://github.com/CleverRaven/Cataclysm-DDA/blob/master/doc/JSON_INFO.md?plain=1#L1835-L1837

Attach save file

N/A

Steps to reproduce

N/A

Expected behavior

Change default evolution scaling to 1 with versioned save migration to quarter it on update
*4 all current base half lifes unless they seemed like they forgot evolution scaling was defaulted to 4
*4 any groups starts/ends that appreciated it was affected by scaling
Remove replace_monster_group/new_monster_group_id/replacement_time code

Screenshots

No response

Versions and configuration

N/A

Additional context

If noone has a good reason why evolution scaling should remain defaulted to 4 or why replace_monster_group/new_monster_group_id/replacement_time isn't entirely superceded by starts/ends I'll do this myself soon

@Procyonae Procyonae added the (S1 - Need confirmation) Report waiting on confirmation of reproducibility label Oct 19, 2024
@PatrikLundell
Copy link
Contributor

If this is an integer (the reference claims its value is one), it could be that they thought the resolution was too coarse if the unit was 1, and so went with 0.25 as the unit. Just a random guess, with nothing beyond it being an integer to support it.

@Procyonae
Copy link
Contributor Author

No it's a float, you can change it between 0.01 and 100 currently at world creation, Harakka found that it was changed to 4 in #14077 which was just a balance change but by changing the default rather than the half lifes or the calc they made the half life values no longer represent days which is what this would seek to revert to make it less confusing/mistake prone for contributors

@RenechCDDA RenechCDDA added <Suggestion / Discussion> Talk it out before implementing and removed (S1 - Need confirmation) Report waiting on confirmation of reproducibility labels Oct 19, 2024
@RenechCDDA
Copy link
Member

Yes huge agree

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Suggestion / Discussion> Talk it out before implementing
Projects
None yet
Development

No branches or pull requests

3 participants