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

Mycus innate dimensional stability #77817

Closed
wants to merge 6 commits into from

Conversation

Kamejeir
Copy link
Contributor

@Kamejeir Kamejeir commented Nov 13, 2024

Summary

Features "Mycus are now dimensionally stabilized"

Purpose of change

It strikes me that Mycus, as much as they want to employ humans, don't account for the world falling apart potentially being very lethal for all their natives turned superspreaders.

The Mycus is, if not close to being, a dimensionally transcendent species (on the Basically Magic end of "technology tree/power level" scale) and would believably have this ability.

Describe the solution

New mutation for the Mycus with the innate effects of a 5-point anchor.

Add conditions to all (tell me if I missed any) checks of DIMENSIONAL_ANCHOR to account for character flags.

(from seeing how this block of code does it, I believe that there isn't really a cleaner solution to be worried about)

{ "not": { "u_has_flag": "BLIND" } },
{ "not": { "u_has_effect": "BLIND" } },
{ "not": { "u_has_worn_with_flag": "BLIND" } }

Describe alternatives you've considered

Testing

I've tested different cases of having a 5-point anchor and Mycus Stability versus a Flaming Eye, then from that working, assume that it will work for all modded interactions of a similar structure.

The same kind of tests have been performed for the C++ based checks, except that I couldn't come up with a way to reliably make something telefrag me.

This being in draft: I think I should try to mutate it naturally first. Also I want to see if there's any way to make the description better.

I got nothing of note to say other than having tested getting the mutation.

Additional context

Yes, this does prevent Mycus people from using the Probability Travel CBM.

image
It's odd that you get extra non-marloss/mycus mutations, but ok.

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies [C++] Changes (can be) made in C++. Previously named `Code` Monsters Monsters both friendly and unfriendly. Mods: Magiclysm Anything to do with the Magiclysm mod Mechanics: Enchantments / Spells Enchantments and spells EOC: Effects On Condition Anything concerning Effects On Condition Mods: Mind Over Matter <Enhancement / Feature> New features, or enhancements on existing json-styled JSON lint passed, label assigned by github actions labels Nov 13, 2024
@github-actions github-actions bot added the astyled astyled PR, label is assigned by github actions label Nov 13, 2024
@SariusSkelrets
Copy link
Contributor

SariusSkelrets commented Nov 13, 2024

Would making the mutation grant an integrated armor with the flags be less intrusive code-wise?

It have the benefit of not needing to change every single Json line in which DIMENSIONAL_ANCHOR is used and not having to do a "or" check every time something checking DIMENSIONAL_ANCHOR is called.

Allowing to directly apply the effect without needing gear does have its benefits even if making that the character and item flags for it are one and the same, both preventing duplication and not needing integrated armor, would be the best choice of the three.

@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Nov 13, 2024
@Kamejeir Kamejeir marked this pull request as ready for review November 14, 2024 14:32
@kevingranade
Copy link
Member

The rationale for this makes no sense. It just makes wild assertions about mycus being dimensionally transcendent (it's not, at all) and that it would "believably have" this ability.

@Kamejeir Kamejeir deleted the mycus-anchor branch November 14, 2024 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [C++] Changes (can be) made in C++. Previously named `Code` <Enhancement / Feature> New features, or enhancements on existing EOC: Effects On Condition Anything concerning Effects On Condition [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Mechanics: Enchantments / Spells Enchantments and spells Mods: Magiclysm Anything to do with the Magiclysm mod Mods: Mind Over Matter Monsters Monsters both friendly and unfriendly. Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants