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

[DNM] Experimentor Refactor and Changes #35466

Closed
wants to merge 15 commits into from

Conversation

DaedalusGame
Copy link
Contributor

@DaedalusGame DaedalusGame commented Feb 8, 2018

🆑
refactor: Refactored and modularized the Experimentor
add: Added some extra reactions for the Experimentor
tweak: The Experimentor generates points now instead of just boosting nodes whatever that does
code: Added the ability to change sounds for the clusterbuster, removed a spawn() and made it possible to clusterspawn objects other than grenades (why)
/:cl:

I only have one more commit to finish this or it will ruin the joke
Another step in the ladder to get rid of passive generation.
Experiment Results are now datumized, so new ones can be (and will be) added easily.
Experiment Results also generate points. They will only generate a meager amount of points, and the amount gained from each experiment result diminishes. The formula is (base_points) / (number of times the result happened ^ 2), so full points for the first time, a quarter for the second, one ninth for the third, etc...

@tgstation-server tgstation-server added Code Improvement Code is now easier to copy paste. Do Not Merge You must have really upset someone Feature Exposes new bugs in interesting ways Refactor Makes the code harder to read Tweak labels Feb 8, 2018
@YakumoChen
Copy link
Contributor

can you still make random battery cells from it or is that gone?

@DaedalusGame
Copy link
Contributor Author

That's the transform reaction iirc, I still have to move it over to the new system.

@epochayur
Copy link
Contributor

but muh timegate

@DaedalusGame
Copy link
Contributor Author

Get the fuckin timegate outta my house

@Bawhoppen
Copy link
Contributor

Finally someone does something with the blood-magic machine. Great stuff

@xenon-difluoride
Copy link
Contributor

You forgot a stage

@tgstation-server tgstation-server added the Merge Conflict Adding upstream files to your repo via drag and drop won't resolve conflicts label Feb 10, 2018
@tgstation-server tgstation-server removed the Merge Conflict Adding upstream files to your repo via drag and drop won't resolve conflicts label Feb 11, 2018
- supermatter
- setting people on fire
- passive effects
- Makes experiment forcing actually work via VV
- Adds cost effects and relic type datums
- Appearance effects now correctly use /datum/relic_effect
@xenon-difluoride
Copy link
Contributor

xenon-difluoride commented Feb 14, 2018

oh I'm dumb the 5th stage missing (at that time) was intentional

…dragonroll

# Conflicts:
#	code/game/objects/items/grenades/clusterbuster.dm
@optimumtact
Copy link
Member

is this ready?

@DaedalusGame
Copy link
Contributor Author

i'll remove the [DNM] and ping a maintainer when it is, shouldn't be long now but people keep asking me to fix bugs in unrelated things

@optimumtact optimumtact closed this Mar 2, 2018
@xenon-difluoride
Copy link
Contributor

y tho

@optimumtact
Copy link
Member

stale

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Improvement Code is now easier to copy paste. Do Not Merge You must have really upset someone Feature Exposes new bugs in interesting ways Refactor Makes the code harder to read
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants