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

Upgraded prisoner transport bus #51891

Merged
merged 9 commits into from
Oct 4, 2021
Merged

Conversation

faefux
Copy link
Contributor

@faefux faefux commented Sep 26, 2021

Summary

Content "Upgraded Prisoner Transport Bus"

Purpose of change

Number 2 in adding and upgrading some vehicles. I decided the prison bus didn't look very much like even an aproximation of a meat weagon, instead seemingly just a regular bus with contraband on.

Describe the solution

  • Added a separate sealed compartment for driver and guards with extra seats, including a reclining chair for sleeping.
  • Added a firearms lock-box in the guards compartment that spawns guns and ammo. (need help with this)
  • Gave the prisoner compartment it's own entrance.
  • Changed seating to the metal benches with shackles I added for pull Prisoner transport van #51645 (prisoner transport van)
  • Removed contraband as it seems very unlikely you are sneaking contraband of any nature on to a sweat box (let alone some of the things in the list.) without plugging it, so unless everyone had chronic diarrhea the last time the vehicle was used..

Describe alternatives you've considered

Remaking it from ground up, making it wider to be more faithful to the images I have seen.
Using a different design as there are many, I have no idea which is more prevalent IRL.

Testing

Tested on old install. Looks fine.

Additional context

I would have liked to add more guns in the box but when trying to add specifically , "count": [ 1, 3 ] to the below line this is incorrect. Could someone advice what I should be using here instead?
"items": [ { "x": 0, "y": 1, "chance": 70, "magazine": 100, "ammo": 50, "item_groups": [ "guns_cop" ] , "count": [ 1, 3 ] } ]

Images:
image
Above: The kind of bus I based it on. But I did not want to add a new set of plastic chairs so instead used to steel.
image
Above: A screenshot of the new version of the bus.

@@ -1791,21 +1789,8 @@
{ "x": -8, "y": 3, "part": "plating_steel" }
],
"items": [
{ "x": -1, "y": 0, "chance": 7, "item_groups": [ "contraband" ] },
{ "x": 0, "y": 0, "chance": 20, "items": [ "id_science_transport_1" ] },
Copy link
Contributor

Choose a reason for hiding this comment

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

Why delete the science ID chance? This was added just a month ago to give some ways to legally enter the new labs.

Copy link
Member

Choose a reason for hiding this comment

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

Specifically this one would be considered to be on one of the guards as these prisoners were being rerouted for experimentation.
{ "x": 0, "y": 0, "chance": 20, "items": [ "id_science_transport_1" ] },

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll give it the same chance of spawning except in the front compartment somewhere if that's okay with you guys. :)

Copy link
Contributor

Choose a reason for hiding this comment

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

It sounds fine to me!

Copy link
Member

Choose a reason for hiding this comment

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

Sounds good. :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added it to the box in the front cabin, @Termineitor244, as well as in the same spot on the prison van but with a smaller chance of spawning. I feel those boxes should both be lock boxes really, (so behind a lockpicking skill-check, i guess) but would this make the card not accessible enough?

Copy link
Contributor

Choose a reason for hiding this comment

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

It would be even better if its behind a lock! It would give even more of a sense of "This is something important, I should keep it with me". And a few locks are never enough deterrent for an experienced player.

@BrettDong BrettDong added [JSON] Changes (can be) made in JSON Vehicles Vehicles, parts, mechanics & interactions labels Sep 26, 2021
@@ -1791,21 +1789,8 @@
{ "x": -8, "y": 3, "part": "plating_steel" }
],
"items": [
{ "x": -1, "y": 0, "chance": 7, "item_groups": [ "contraband" ] },
{ "x": 0, "y": 0, "chance": 20, "items": [ "id_science_transport_1" ] },
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
{ "x": 0, "y": 0, "chance": 20, "items": [ "id_science_transport_1" ] },
{ "x": 0, "y": 0, "chance": 20, "items": [ "id_science_transport_1" ] },

@Maleclypse
Copy link
Member

Let me know once you are ready for review?

…asing. Ironically almost forgot the original reason I even considered a public PR for them both!
@faefux
Copy link
Contributor Author

faefux commented Sep 29, 2021

Let me know once you are ready for review?

There you go, @Maleclypse , sorry I take so long and thank you for being patient with my multiple git-fails. :)

@faefux
Copy link
Contributor Author

faefux commented Oct 1, 2021

sigh I'll give this thing one more chance if someone can show me where exactly there is an issue here is with the failing check. reeeee- :(.

@kevingranade
Copy link
Member

The failing test is not actually your problem, this looks good to go with a final review.

Copy link
Member

@Maleclypse Maleclypse left a comment

Choose a reason for hiding this comment

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

Thank you very much for doing this. It's pretty cool.

@faefux
Copy link
Contributor Author

faefux commented Oct 2, 2021

The failing test is not actually your problem, this looks good to go with a final review.

Ah, sorry for whining so then, I am used to it being otherwise. Merci!

Thank you very much for doing this. It's pretty cool.

Appreciated, its just a small little thing. :) Hopefully I will improve at this to upload some larger more fun and cool things eventually! :)

@ZhilkinSerg ZhilkinSerg merged commit cd0be9d into CleverRaven:master Oct 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[JSON] Changes (can be) made in JSON Vehicles Vehicles, parts, mechanics & interactions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants