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

Frog and Snail Mutation Trees #51630

Merged
merged 108 commits into from
Sep 24, 2021

Conversation

Maleclypse
Copy link
Member

@Maleclypse Maleclypse commented Sep 15, 2021

Summary

Content "Adds Frog and Snail Mutation Trees"

Purpose of change

Adds a few new mutation trees. Fixes #51569 .

Describe the solution

Adds mutations, thresholds, mutagen, recipes, spawns, and unique mutagen ingredients.

Describe alternatives you've considered

None

Testing

Ongoing

Additional context

Maleclypse and others added 30 commits March 26, 2020 22:57
August 1 2020 Catchup to Master
Catchup for Artifact conversions
@Maleclypse Maleclypse force-pushed the More-Mutation-Trees branch 3 times, most recently from e16b2a6 to f610a15 Compare September 19, 2021 03:36
Maleclypse and others added 2 commits September 18, 2021 22:36
fixes

Apply suggestions from code review

Update data/json/mutations/mutation_spells.json

Apply suggestions from code review

Apply suggestions from code review

Lint

Update mutations.json

Apply suggestions from code review

Update mutations.json

Need to add rabbit dreams

Fixes

Additional Insect and Cattle traits

Merge fixes, rabbit dreams, eoc

Update mutations.json

Update mutations.json

Co-Authored-By: actual-nh <74678550+actual-nh@users.noreply.github.com>
@Maleclypse
Copy link
Member Author

maleclypse — Today at 10:00 PM

            //Collect valid time of day ranges
            if( elem == "DAY" || elem == "NIGHT" || elem == "DUSK" || elem == "DAWN" ) {
                if( elem == "DAY" ) {
                    valid_times_of_day.emplace_back( sunrise, sunset );
                } else if( elem == "NIGHT" ) {
                    valid_times_of_day.emplace_back( sunset, sunrise );
                } else if( elem == "DUSK" ) {
                    valid_times_of_day.emplace_back( sunset - 1_hours, sunset + 1_hours );
                } else if( elem == "DAWN" ) {
                    valid_times_of_day.emplace_back( sunrise - 1_hours, sunrise + 1_hours );
                }
            }

I want to add Night and Day as active conditions in enchantments
if( active_conditions.second == condition::NIGHT ) { return guy.pos().z < 0; }
I get that I need to change the return guy.pos().z < 0; but I'm not sure how to pull into the day ranges that I think are elem

maleclypse — Today at 11:01 PM

Is it     if( active_conditions.second == condition::NIGHT ) {
        return is_night;
    }```  
?
and to get that to work do I need to include the calendar.h?

@Maleclypse
Copy link
Member Author

failed tests

These tests are failing but when I look at the tests folder I can't see why they would be failing now.

@Maleclypse
Copy link
Member Author

Well now I have to figure out how the points are awarded to a given mutation. Ugly somehow awards 24 Batrachian points by itself.

@Maleclypse Maleclypse marked this pull request as ready for review September 24, 2021 13:12
@kevingranade kevingranade merged commit 75de031 into CleverRaven:master Sep 24, 2021
"You have a strange dream about being the Easter Bunny.",
"You have a comforting and warm dream of living as a rabbit."
],
"category": "LIZARD",
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't this be "RABBIT"?

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks! This was already merged so someone will have to create a new PR with the fixes. Let me know if you want to that and if not I'll put them on my todo list.

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh, then I will do the PR, serves me as an excuse to update my master, just a question, NIGHT_RAIDER is supposed to have the same description as TUNNEL_FIGHTER? To know if I should change it too.

Copy link
Member Author

Choose a reason for hiding this comment

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

no it should have different text . I just forgot to update it.

Copy link
Contributor

Choose a reason for hiding this comment

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

Done #51859.

"copy-from": "iv_mutagen_flavor",
"type": "COMESTIBLE",
"name": { "str": "batrachian serum" },
"description": "A super-concentrated mutagen with that looks like pond water… are the shadows in it moving? You need a syringe to inject it… if you really want to?",
Copy link
Contributor

Choose a reason for hiding this comment

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

"A super-concentrated mutagen with that looks"
I think the "with" should be removed... Or maybe I'm just bad at English?

Copy link
Contributor

Choose a reason for hiding this comment

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

Seconding, the with should be removed.

"copy-from": "mutagen_flavor",
"type": "COMESTIBLE",
"name": { "str": "batrachian mutagen" },
"description": "A shadowy mutagen that looks like might be full of tiny swimming tadpoles.",
Copy link
Contributor

Choose a reason for hiding this comment

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

"A shadowy mutagen that looks like might"
Like "it" might?

"id": "NIGHT_RAIDER",
"name": { "str": "Night Raider" },
"points": 3,
"description": "You know how to fight in cramped tunnels at risk of collapse. You gain bonuses when fighting underground.",
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this supposed to have the same description as "TUNNEL_FIGHTER"?

@@ -568,7 +670,7 @@
"id": "TERRIFYING",
"name": { "str": "Terrifying" },
"points": 1,
"description": "There's something about you that creatures find frightening, and they are more likely to try to flee.",
"description": "There's something about you that humans and post-humans find frightening, and they are more likely to try to flee.",
Copy link
Contributor

Choose a reason for hiding this comment

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

This affects the moral of all creatures:

if( u->has_trait( trait_TERRIFYING ) ) {
            effective_morale -= 10;
        }

(From src/monster.cpp)
So "creatures" is more appropriate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Map / Mapgen Overmap, Mapgen, Map extras, Map display Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies Spawn Creatures, items, vehicles, locations appearing on map
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Suggestion: Frog Mutation