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

fix unable to research ai tech using ruin type rnd console #22878

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

warface1234455
Copy link
Contributor

@warface1234455 warface1234455 commented Dec 21, 2024

i forgot about this because back then ai tech was still a testmerge thing

Testing

image

Changelog

🆑

bugfix: fix unable to research ai tech using ruin type rnd console
bugfix: fix mech fab ruin type unable to be rebuilt
tweak: Allows rack creator to be able to link to ruin research tech by multitooling the rack creator board
tweak: rack creator no longer requires beakers because it doesn't store any chems
/:cl:

@Yogbot-13 Yogbot-13 added the Fix This fixes an issue. Please link issues in fix PRs label Dec 21, 2024
Copy link
Contributor

@ToasterBiome ToasterBiome left a comment

Choose a reason for hiding this comment

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

This lets ruin people use the station's AI research. Also conversely lets station people use ruin's. Something that could be done is making a ruin rack creator.

@warface1234455
Copy link
Contributor Author

This lets ruin people use the station's AI research. Also conversely lets station people use ruin's. Something that could be done is making a ruin rack creator.

no it doesn't use normal station ai research, i have made a seperate techs

/obj/machinery/rack_creator/RefreshParts()
var/obj/item/circuitboard/machine/rack_creator/rack_creator = circuit
if(rack_creator.unlocked)
stored_research = SSresearch.ruin_tech
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

Doesn't this mean that it's shared between all neutral factions, even ones which might oppose each other (e.g. free golems and free miners)?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

well the tech is learned in every factions outside from NT so yeah

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix This fixes an issue. Please link issues in fix PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Neutral RnD setups are unable to research AI
4 participants