Skip to content

Solving the nucleus paradox#6595

Open
domin2ktr wants to merge 60 commits intoRevolutionary-Games:masterfrom
domin2ktr:Nucleus-buff
Open

Solving the nucleus paradox#6595
domin2ktr wants to merge 60 commits intoRevolutionary-Games:masterfrom
domin2ktr:Nucleus-buff

Conversation

@domin2ktr
Copy link
Contributor

@domin2ktr domin2ktr commented Dec 21, 2025

Brief Description of What This PR Does

This PR reduces the cost of osmoregulation and basic movement by 25%, thereby solving the problem of evolutionary nucleus cost.

  • PR author has checked that this PR works as intended and doesn't
    break existing features:
    https://wiki.revolutionarygamesstudio.com/wiki/Testing_Checklist
    (this is important as to not waste the time of Thrive team
    members reviewing this PR)
  • Initial code review passed (this and further items should not be checked by the PR author)
  • Functionality is confirmed working by another person (see above checklist link)
  • Final code review is passed and code conforms to the
    styleguide.

@revolutionary-bot
Copy link

The lead programmer for Thrive is currently on vacation until 2026-01-07. Until then other programmers will try to make pull request reviews, but please be patient if your PR is not getting reviewed.

PRs may be merged after multiple programmers have approved the changes (especially making sure to ensure style guide conformance and gameplay testing are good). If there are no active experienced programmers who can perform merges, PRs may need to wait until the lead programmer is back to be merged.

@domin2ktr domin2ktr mentioned this pull request Dec 21, 2025
3 tasks
osmoregulationCost *= 20.0f / (20.0f + colonySize);
}

// 30% osmoregulation bonus if have nucleus
Copy link
Contributor

Choose a reason for hiding this comment

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

This comment is not needed (and it's 25% not 30%)

Copy link
Contributor

@Accidental-Explorer Accidental-Explorer left a comment

Choose a reason for hiding this comment

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

I know this is currently not being worked on, so I will just note this for future reference:

Before this could be merged, the tooltips for organelles in the editors would have to correctly be updated to show their new osmoregulation cost, just as happens when you change which membrane you have.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: In progress

Development

Successfully merging this pull request may close these issues.

5 participants