Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

Conversation

@maxion12345
Copy link
Contributor

@maxion12345 maxion12345 commented Sep 5, 2021

Github documenting your Pull Request

This adds the General Population system to Yogurtstation13! Included in a new brig interface with preset crimes, and a new way to keep prisoners secured!
Heres a photo of the interface.

As a demonstration, I will brig Brodie Brinigh for Theft and Trespass. We'll input the details, and it sets the time.

Hitting "Finalize ID" prints out the completed prisoner ID, and begins tracking our time in the brig. It also prints a piece of paperwork to be filed by the warden (or thrown away). The ID looks something like this:

Next, we'll need to strip our prisoner. Find an un-assigned GenPop locker, open it, and strip the prisoner of everything but his jumpsuit (ID included)

To seal the locker, slide the ID over the locker and it locks shut and assigns itself to that ID. It can be opened by the prisoner later once the brig time is up. Then just put the prisoner into a cell with the temporary ID, and you are good to go! The system automatically updates arrest status and security records.

Once released, special access is added to the prisoner ID card that enables them to walk out of turnstiles, so they can collect their stuff and let themselves out. This feature isn't pictured as turnstiles haven't been added into the game yet.

NEW MAPS (turnstiles to be added later)
Yogurtstation: Upgraded cells with genpop

Meta: has basically the same layout as yogurtstation

Kilostation: just changed the terminals and lockers to gen
pop and added flasher button under em

Delta: Only two cells so no room for genpop

Wiki Documentation

Update brig photos
Update brigging procedure to brig with this system instead of the old one
New procedure:

  1. Take the prisoner to the brig and inform the Warden of their crimes so their Security Record may be updated.
  2. Take the prisoner to a Genpop Interface, input crimes, and print the ID.
  3. Find an empty genpop locker, open the locker and hold the prisoner over it.
  4. Empty their pockets and remove their gloves, backpacks, tool belt, gas masks, and any flash resistant equipment such as Sunglasses, Welding Masks/Goggles, Space Helmets, etc. Make sure the prisoner is clothed once you have completed this step, but ensure to remove their ID.
  5. Search the items removed and be sure to check the internals box in their backpack.
  6. Confiscate any contraband and/or stolen items, as well as any tools that may be used for future crimes.
    These are to be placed in evidence, not left on the brig floor or your personal use, until they can be returned to their rightful owners.
  7. Close the locker and lock it by swiping the Genpop ID over it.
  8. Put ID onto prisoner, and place them within a cell.

Changelog

🆑 NSV13 & Maxim
rscadd: Added the General Population brigging system, complete with automatic time setting and group cell capabilities.
/:cl:

@maxion12345 maxion12345 requested a review from a team as a code owner September 5, 2021 23:44
@Yogbot-13 Yogbot-13 added DME Edit This PR affects the yogstation.DME file Feature This adds new content to the game Javascript This PR contains javascript code. Mapping Will result in a change to a map. labels Sep 5, 2021
@maxion12345 maxion12345 marked this pull request as draft September 5, 2021 23:44
@maxion12345
Copy link
Contributor Author

maxion12345 commented Sep 5, 2021

whoop fixed it all we're golden now

@github-actions github-actions bot added the tgui This PR affects TGUI label Sep 5, 2021
@maxion12345
Copy link
Contributor Author

@Chubbygummibear
Copy link
Contributor

This is rad, love seeing slick and intuitive UIs with tgui.

@Tipy1802
Copy link

Conflicted on this, while it does seem helpful, wouldn't it put the warden out of a job?

@maxion12345
Copy link
Contributor Author

did it work

@maxion12345
Copy link
Contributor Author

this is hell man all i wanted to do was fetch upstream

@github-actions github-actions bot added the Github changes to the .github folder label Sep 16, 2021
@maxion12345
Copy link
Contributor Author

ok we are back where we started
fuck

@maxion12345
Copy link
Contributor Author

new plan
we don't touch maps until turnstiles are merged and we can do it all at once because every time I fetch upstream it cuts yogstation in half and causes massive merge conflicts

@maxion12345
Copy link
Contributor Author

time to go fix random conflict

@maxion12345
Copy link
Contributor Author

reverted maps so I can pull master to fix conflicts, re-making them now. I'll also take this time to look into that bug.

@maxion12345
Copy link
Contributor Author

Development of this will be stopped for a few days, I'll be away from spaceman's. The test merge has been informative so far, and I've noted somethings for further development, thank you for any feedback

@burn1cycle
Copy link

Feature has been a disaster.
There's no safe way to put people in genpop when it's full or if the prisoners are violent it's also dangerous. What I've observed is security avoiding brigging to avoid having to use genpop. I have yet to see a command or sec regular whose enthused about it however it's getting positive reviews from shitters who break spacelaw and ruin the round for others.

@maxion12345
Copy link
Contributor Author

after gathering some feedback from players, I believe the main issues of this system revolve around the big gen pop cell. Namely:

  1. Getting people into the cell is a pain when there are others already inside
  2. Prisoners are assholes and beat each other up instead of just coloring, playing arcade, and balling
  3. due to the nature of it being a cell, it's difficult to actually break up these fights

Most feedback thinks the new brig system is actually good, and it all falls apart when it comes time to put the prisoners into a cell without prisoners. With that in mind, I am going to be removing the big general cell, and instead of having only one cell per prisoner, as it was before. Delta and Kilo already use this, if you want an example of how it'll look. I'll take this time to spruce up the brig cells as well, because I still want the prisoners to have something to do, even if it's just read space law. maybe I'll give them a shitter.

@maxion12345
Copy link
Contributor Author

taking the time to merge master so there are no conflicts later down the line

@maxion12345
Copy link
Contributor Author

figured out that the timer was running slow, with each 'second" taking place over two real seconds. It'll update by 2 seconds to compensate. No more absurdly long timers! also added animal cruelty.

@burn1cycle
Copy link

burn1cycle commented Oct 1, 2021

Is it a simple or complex effort to incorporate the computer records database entered crimes?

I often enter crimes in on the Sec HUD or Computer Records and genpop has no interaction with this and it would be nice to have it transfer over quickly with a wave of the soon-to-be prisoner's ID(their role issued ID) onto the Prisoner Management Interface and have a prisoner ID come out if the crimes were set in the Database(or if capital crime erroring them to check with HoS/Captain). This way I don't have to do it twice.

YOU DONT KNOW YOUR FUCKING SEC TIME #12387 is a problem I have too.
Adjacent to the above, it would be nice if it printed the station time that the prisoner gets out in and even says "You feel so bored you do the math in your head and its XX:XX minutes/seconds until you get out".

Other issue I have is the single panel glass isn't as strong as full tile and can be more easily broken through fire, pressure, and brute. Can we get a buff in the numbers to put them on par with full tile reinforced glass?

I have liked the improvements to brigging a lot. It's much easier and feels a lot less archaic. Seriously our old brig system was OLD, I don't think it changed in years and it felt like it it was a stiff and clunky piece of crud. New system is sleek and shiny.

@maxion12345
Copy link
Contributor Author

Is it a simple or complex effort to incorporate the computer records database entered crimes?

I often enter crimes in on the Sec HUD or Computer Records and genpop has no interaction with this and it would be nice to have it transfer over quickly with a wave of the soon-to-be prisoner's ID(their role issued ID) onto the Prisoner Management Interface and have a prisoner ID come out if the crimes were set in the Database(or if capital crime erroring them to check with HoS/Captain). This way I don't have to do it twice.

YOU DONT KNOW YOUR FUCKING SEC TIME #12387 is a problem I have too. Adjacent to the above, it would be nice if it printed the station time that the prisoner gets out in and even says "You feel so bored you do the math in your head and its XX:XX minutes/seconds until you get out".

Other issue I have is the single panel glass isn't as strong as full tile and can be more easily broken through fire, pressure, and brute. Can we get a buff in the numbers to put them on par with full tile reinforced glass?

I have liked the improvements to brigging a lot. It's much easier and feels a lot less archaic. Seriously our old brig system was OLD, I don't think it changed in years and it felt like it it was a stiff and clunky piece of crud. New system is sleek and shiny.

As long as the prisoners name is properly set, it'll automatically add crimes to their sec record. In regards to the ID issue I've not been able to reproduce either on main or private server, so I'm not sure if it's an unrelated bug or not. Maybe it's byond, dunno.

I think most of the issues were handled with the change from a big cell back to three small ones. The ID part gets people confused at time, and I am considering seeing if I can apply the code from this to the old timers to give it similar functionality without all the fumbling with a special prisoner ID.

@maxion12345
Copy link
Contributor Author

Closing in favor of #12422

may re-open if testmerge for that one goes badly

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

DME Edit This PR affects the yogstation.DME file Feature This adds new content to the game Github changes to the .github folder Javascript This PR contains javascript code. Mapping Will result in a change to a map. tgui This PR affects TGUI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants