-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Amphibian Additions - Final Frogs #73202
Conversation
They always should have had this, I think it was an oversight on my part
-Move baby flags to abstract to reduce JSON size -Restore leap special to spring peepers and vocal frogs
I'll make these into a separate PR after this one.
My first mapgen effort, to ensure pickerel frogs actually have a chance of appearing in caves.
Made it more apparent that the tail_tadpole is intended to have land movement. I also lowered their speed to account for this
Spell checker encountered unrecognized words in the in-game text added in this pull request. See below for details. Click to expand
This alert is automatically generated. You can simply disregard if this is inaccurate, or (optionally) you can also add the new words to Hints for adding a new word to the dictionary
|
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.
Auto-requesting reviews from non-collaborators: @Termineitor244
Just a heads up that this has sparked a fair bit of conversation about what how much detail we should be supporting in monsters with our current stats. I don't think you need to change this PR, it looks good and you've clearly worked hard and nothing in it is inappropriate. however, I'd like to ask that you hold off on further big additions of cosmetic creature variants until we've had a chance to determine what the best way to manage this kind of content is, because I don't think our current systems for it work really great. It winds up being a lot of lines of JSON to manage, when taken as a whole. In a larger sense, we've needed a system for monster variants for a long time, and these are really starting to get down to the heart of it. there are a lot of lines of monstergroup information here that basically amount to different spawning weights for variants of the "frog" abstract, with creatures that are functionally just a reskinned frog. If you're able to see a way to streamline that with existing tools it'd be cool but in the big picture I think we need to push for some new infrastructure for that, and move stuff like frogs into it. until we have that, I think we should hold off on any further high-detail variants of creatures like this. I'll post an issue with more details. |
I've noticed the json bloat growing myself, so right there with you. This is the last big addition like this I had planned. I have mentioned salamanders but they are very much an "at some vague future date" addition. I won't do anything with those until a proper variants system is in place (even then they may need to be abridged, there's even more species of them than frogs).
After this PR I'm turning towards polishing what I've added. I can definitely do some cleanup/json reduction with groups. I have an idea of how to remove half of the eggs and tadpoles, but I'll need to check if it functions the way I think it does. |
Summary
Content "Adds the last remaining natural frog species native to New England."
Purpose of change
Round out the remaining frogs of New England.
Bring them to parity with the existing creatures by including a few mutant varieties.Describe the solution
Adds new creatures, adult frogs and tadpoles, for the remaining natural species common to New England:
Additionally adds mutants:I'll be implementing these in their own PRfrom Mink Frogs, the Tailed Frog becomes the Musk Frog; big mutants that nest in muddy burrows created with their tailsfrom Pickerel Frogs, the Palette Frog becomes the Painter Frog; big-ish mutant that sheds poisonous fluids of bright warning colorsfrom Green, Wood, and Leopard Frogs, the Many-Legged Frog becomes the Skittering Frog, and then a Wormhost FrogDescribe alternatives you've considered
Settling for fewer frogs
Testing
Did some hours of flyover spawn testing. All new monsters appeared through normal map gen.
Tested hatching/growing/laying in large batches. Everything checked out, many-legged frogs are appropriately rare.
Tested dissection and butchery for each monster.
Tamed all frogs with Tamable Wildlife.
Additional context
6th implementation of #67029 which I sorely need to update
Will need to be merged after #72413This includes the last new frogs from New England but won't be my last frog-related PR. I have a list of things to polish before I consider moving onto salamanders. I also have a few more oddball frog mutants that may come down the road.