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

Research Trait fixes #4195

Closed
wants to merge 3 commits into from
Closed

Conversation

Diegoflores31
Copy link
Contributor

@Diegoflores31 Diegoflores31 commented Aug 17, 2023

About the pull request

ALL traits will now use Potency instead of level to calculate their effects . I have noticed there was some traits that were not using this and as a result they were not working at all when used with modifiers like Boosting. This PR corrects that behaviour to work as it should .

PD : STATS UNTOUCHED THIS PR JUST FIXES THE BEHAVIOUR ALL NUMBERS ARE THE SAME AS BEFORE .
PD 2 : potency = level / 2 which means i have to add a (potency * POTENCY_MULTIPLIER_MEDIUM) to keep the numbers the same

Explain why it's good for the game

bug bad . research = funny

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
fix: Fixes Booting , Cryometabolizing , and other metabolites not working on some traits
/:cl:

@github-actions github-actions bot added the Fix Fix one bug, make ten more label Aug 17, 2023
@Weuyn
Copy link
Contributor

Weuyn commented Aug 17, 2023

Allowing boosting to affect ciphering is a terrible idea, ciphering costs many times times as much to amplify to make it harder to spam hives (it had a unique modifier that no other property has, except the egg plasma one iirc).

Amplifying boosting from 1 to 5 costs only 7 points. Amplifying ciphering from 1 to 5 costs like 40? I think? It would be really funny, but not balanced.

fire/explosive mod traits don't care about boosting anyway, boosting only has an effect when metabolized?

@Diegoflores31
Copy link
Contributor Author

Allowing boosting to affect ciphering is a terrible idea, ciphering costs many times times as much to amplify to make it harder to spam hives (it had a unique modifier that no other property has, except the egg plasma one iirc).

only corrupted are allowed to be friendly at marines anyways so spamming hives is not really concern . and boosting is not something you get every round . its almost as rare as fire penetrating and its outright dumb that such a high tier, hard to get modifier just doesnt work with like half of traits

@Huffie56
Copy link
Contributor

would it fixed this issue @Diegoflores31 ?
#3894

@Diegoflores31
Copy link
Contributor Author

would it fixed this issue @Diegoflores31 ? #3894

No . That issue is not a bug sadly it is "intended" because of how research warcrimes if anything i will rather do something about it on a different PR

@Unknownity
Copy link
Contributor

This should require clarification from a developer regarding Boosting not affecting some properties whether that is actually intended or not.
@Segrain

@Diegoflores31
Copy link
Contributor Author

Well the description clearly says that boost other properties . And wiki says the same . Its counter intuitive that it just applies to some chems and not to others

@Diegoflores31
Copy link
Contributor Author

i realized i did a lil bit of shitcode will rewrite this PR tomorrow

@Segrain Segrain self-requested a review August 29, 2023 17:22
@Diegoflores31 Diegoflores31 deleted the research branch October 30, 2023 05:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Fix one bug, make ten more
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants