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

Adds pathology goggles and gives pathologists access to pharmacy #3233

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

Conversation

RikuTheKiller
Copy link
Contributor

@RikuTheKiller RikuTheKiller commented Sep 4, 2024

About The Pull Request

Adds viral analyzer goggles, i.e. pathology goggles.
It's a replacement for the plain old science goggles they had previously.
Pathologist now spawns with them by default and maps have been adjusted. (did it through code, tell me if it's an issue)

Additionally, the new goggles have an item action to toggle the viral analyzer aspect on and off.
The pathology goggles have the reagent scanning of science goggles, but not the item/machine scanning.
Their addition also includes a refactor for the virusView() proc into a trait + element combo.

Also gives pathologists access to pharmacy regardless of skeleton crew status.

Why It's Good For The Game

Non-pathologists shouldn't have the ability to scan for pathogens roundstart.
Science goggles already had 2 features even before viral scanning was added to it, so it needed to get split.
The toggle action helps with the absolute visual overload when you have airborne diseases and such.

Pathologist now relies on extremely niche chems that no chemist (rightfully) knows how to make.
There's also an existing precedent for exceptions like this, since MD can access pharmacy as well.

Changelog

🆑
add: New pathology goggles with a nifty toggle for the pathogen visuals. No more visual overload!
balance: Pathologists now have access to pharmacy, in light of their new dependence on obscure chems.
balance: Science goggles can't see pathogens anymore. You're more reliant on the pathologist again.
fix: Virus view now works at roundstart. I fixed it by just refactoring it into something sensible.
/:cl:

@ophaq
Copy link
Contributor

ophaq commented Sep 5, 2024

Good feature. Hated how I couldn't make chems round start for pathology without getting let into pharma first. Also the science goggles let even chemists see pathogens which seemed kinda weird so glad that got split up.

Now we just need pathogens to actually increase faster in strength when applying virus plasma because that is ultra slow.

@RikuTheKiller RikuTheKiller marked this pull request as ready for review September 5, 2024 23:08
@ThePooba
Copy link
Collaborator

@RikuTheKiller mergeconflicxts

@RikuTheKiller
Copy link
Contributor Author

@RikuTheKiller mergeconflicxts

Yay ill fix them later today

@RikuTheKiller
Copy link
Contributor Author

@RikuTheKiller mergeconflicxts

I have redefined today as tomorrow lmao

@RikuTheKiller
Copy link
Contributor Author

okay something fucked up in the merge process
like, badly

@Absolucy
Copy link
Member

will tm if conflicts are fixed

@Absolucy Absolucy added the Stale label Nov 8, 2024
@Absolucy
Copy link
Member

Absolucy commented Nov 8, 2024

merge conflicts + inactivity

@Absolucy Absolucy removed the Stale label Nov 8, 2024
@Kitsunemitsu Kitsunemitsu added Tweak change a thing to work a little different Balance perfectly balanced as all things should be labels Dec 9, 2024
@DexeeXI
Copy link
Collaborator

DexeeXI commented Dec 16, 2024

Conflicted, Stale.

@DexeeXI DexeeXI added the Stale label Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance perfectly balanced as all things should be Merge Conflict Stale Tweak change a thing to work a little different
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants