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

Medical tracker implant #1984

Closed

Conversation

Radezolid
Copy link
Contributor

About the PR

This PR adds the medical tracker implant, pretty much a tracker implant with sensors on Vitals and not on Coordinates, but for medical and civilian use.
It can be printed at the medical techfab at the cost of 10 steel sheets and 10 glass sheets (Double the empty implanter).
Works best with #1983

Why / Balance

This tracker aims to reduce or at least help with the salvage deaths that end on round removal on the asteroid or wrecks, note that this doesn't help if the person is drifting and it doesn't have the coordinates configuration, so you can't track them on the crew monitor, encouraging antags to grab it.

Technical details

Creates a new implant identical to tracker, but with SensorVitals instead of SensorCords and sends the rattle on the Medical radio channel.

Media

2024-10-07.19-53-27.mp4

Requirements

Breaking changes

Changelog

🆑

  • add: Added a new medical tracker printable at the Medical TechFab which only relays Vitals sensors and it will notify medical if you go in critical state but it alone won't show the position on the crew monitor.

@Radezolid Radezolid requested a review from a team as a code owner October 7, 2024 23:45
@Radezolid
Copy link
Contributor Author

If any ideas on how to change the Implanter option from the medfab to show something like "medical tracker" please let me know.
Also if this should be a cargo crate let me know and I'll do changes to it, more than anything because of how this may affect kill targets on medical personnel.

@Radezolid Radezolid closed this Oct 8, 2024
@Radezolid
Copy link
Contributor Author

Been pointed out about possible powergaming with this so i'll hold off until i find a way to ensure it isn't powergamed.

@Radezolid
Copy link
Contributor Author

Me when forget that draft is a thing.

@Radezolid Radezolid reopened this Oct 8, 2024
@Radezolid Radezolid marked this pull request as draft October 8, 2024 22:13
@deltanedas
Copy link
Member

command has always powergamed it with tracking implants (which tell sec, much more useful for catching the valids)

@Radezolid
Copy link
Contributor Author

command has always powergamed it with tracking implants (which tell sec, much more useful for catching the valids)

Still, the main reason was to make new salvies not get RR due to mistakes (even if it's just skill issue) or lag in a wreck.

I've tought of:

  1. Give it back into the lathe but make it stupid expensive.
  2. Make it buyable on logi.
  3. Make Salvage spawn with one in their bag (This might have issues as i've spawned as thief salvage and dropped the beacon on spawn).
  4. Give the CMO two of them in their locker.
  5. Lock it behind research, only printable at medfab but expensive.

But none of those really convinces me, since:

  1. Logi can just buy mats and spam them.
  2. Logi will powergame the shit out of it and only them have trackers.
  3. Maybe? Feels a bit bad that in the end only salvage can get them.
  4. They wont ever give them to people that needs them and they will give it to command.
  5. Good fucking luck with this, by the time they research it salv got gibbed due to rot in space.

@deltanedas
Copy link
Member

if you add it as an orderable item people will just buy regular trackers instead since they have coords

@Radezolid
Copy link
Contributor Author

Can't find a viable way of making it so people don't powergame it.

@Radezolid Radezolid closed this Oct 15, 2024
@Radezolid Radezolid deleted the Medical-vitals-monitor branch October 15, 2024 00:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: YML Changes any yml files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants