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

[WIP] Reworks the RUST #162

Draft
wants to merge 8 commits into
base: dev
Choose a base branch
from
Draft

[WIP] Reworks the RUST #162

wants to merge 8 commits into from

Conversation

Cupax3
Copy link
Contributor

@Cupax3 Cupax3 commented Oct 18, 2021

About the Pull Request

See title.

The new React() proc features a smaller code-footprint and hopefully improved readibility.
The current new features include support for decaying reactions as well as generic handling of required reactants and products.
Also added were maxima for both the temperature and energy level of the reactor for reactions to happen.
An improved, but still not complete, distribution system of reactants to reactions should help with making complex systems less random and more probabilistic.

Planned for current additions:
Fit distribution system to a distribution plot making reactions with more reactants present more likely.
Add automatic decay once reactant hits a certain amount to prevent overload.
Add an example implementation of the CNO-Cycle.

Planned:
Rework the entire reactant addition and subtraction system to make it semi-tiered and involve multiple departments for high-end stuff.
Remove the lag monster known as gyrotron.
Rework attached machinery to also require more involvement.
Rework science progression to lock high-level stuff behind new materials.
New materials.
Make the RUST actually radioactive
More to be added.

Please add your suggestions in the comments!

Code review appreciated but please bear in mind this is still in-dev.

Why It's Good For The Game

[TBD]

Did you test it?

Yes - WIP

Changelog

[TBD]
🆑
/:cl:

See title.

The new React() proc features a smaller code-footprint and hopefully improved readibility.
The current new features include support for decaying reactions as well as generic handling of required reactants and products.
Also added were maxima for both the temperature and energy level of the reactor for reactions to happen.
An improved, but still not complete, distribution system of reactants to reactions should help with making complex systems less random and more probabilistic.

Planned for current additions:
Fit distribution system to a distribution plot making reactions with more reactants present more likely.
Add automatic decay once reactant hits a certain amount to prevent overload.
Adds a boatload of comments.

Also updates old reactions.

Also adds support back for special handling, woups.

Removes unused helpers that didn't wound up necessary.

Adds a comment to a debug function to have it be removed before production merge.

Also makes checks not be angy by removing .Find because for some reason that's a macro whatever.
@Shadowtail117 Shadowtail117 added ⚖1 Balance Changes to the balance of the game. 🏆9 Achievement earned "OriginalNaym wants to merge 0 commits" 📚2 Needs Wiki Update This PR requires contents of a wiki page to be updated. 🔨1 Tweak A small change/addition to already existing systems. 🔬1 Code improvement Code is now easier to copy paste. labels Oct 18, 2021
Haha yeah "Tested"
@Cupax3
Copy link
Contributor Author

Cupax3 commented Oct 18, 2021

Forgot to mention this but I'm keeping the super simple key-string-based reactant system because it's so damn easy to modify it's a plus rather than a negative. Seriously, admins, this is bliss to work with to mess with the reactants

https://en.wikipedia.org/wiki/Stellar_nucleosynthesis

Also does miscellanious cleanup and adds a hopefully better sorting of reactions
This is probably going to irradiate the entire Torch
Also fixes a bug where reactants wouldn't be properly added to the copy. 4 hours......
@Cupax3Bot
Copy link
Collaborator

Test Merge Deployed

Server Instance

TeguStation 12

Merged By

Admin

Revision

Origin: c27735b
Pull Request: d018717
Server: c1bec07b95757f65200db6e7e6f73c6edbdac988

@Cupax3Bot
Copy link
Collaborator

Test Merge Updated

Server Instance

TeguStation 12

Merged By

Admin

Revision

Origin: c27735b
Pull Request: 2229d5e
Server: e3dff1a582bb57a0bf80d59640275c46bca8f581

@Cupax3Bot
Copy link
Collaborator

Test Merge Updated

Server Instance

TeguStation 12

Merged By

Egor

Revision

Origin: 245751a
Pull Request: 2229d5e
Server: 473bfefb1da732cc958369ddccf31202c0707e94

@Cupax3Bot
Copy link
Collaborator

Test Merge Removed

Rads rads we're the rads
@Shadowtail117
Copy link
Collaborator

CI is failing on global hash

@Cupax3
Copy link
Contributor Author

Cupax3 commented Oct 19, 2021

CI is failing on global hash

And it will probably continue to fail as I modify globals with new features

Explosions can now be rather destructive. Like, really destructive.
@Cupax3 Cupax3 force-pushed the fusion branch 5 times, most recently from 6a2cb3f to a148b4a Compare October 20, 2021 03:16
Haha frick the brig
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚖1 Balance Changes to the balance of the game. 🔬1 Code improvement Code is now easier to copy paste. 🔨1 Tweak A small change/addition to already existing systems. 📚2 Needs Wiki Update This PR requires contents of a wiki page to be updated. 🏆9 Achievement earned "OriginalNaym wants to merge 0 commits"
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants