-
Notifications
You must be signed in to change notification settings - Fork 605
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
Updated the NSF Paladin to have a mixed atmos system & a few other tweaks. #2712
base: master
Are you sure you want to change the base?
Conversation
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Thanks for polishing up this shuttle! However there is no reason I can think of for it to not have the appropriate atmos setup. That is a mapping standard and I cannot think of any reason why this shuttle should be granted an exception. The setup that you showed me before with the appropriate mixer looked fine. I'd love to do a walk through but it seems like that point should be addressed first. @Leander-0 is there a reason that you feel that this shuttle should be an exception? |
i mean it was originally accepted without it but the reasoning against the change is that another canister with a separated atmos system would take a little more space of an already small shuttle that it is used mostly for assault more than as a transport or cell shuttle, the premixed air canister already does what the extra canister and mixer would do. the pilot should only need to change 1 canister instead of having to handle 2 if they want another type of gas inside the shuttle. |
The two canisters is part of a Frontier wide shuttle compliance effort to ensure that species with specific atmospheric needs are provided with available gasses. The additional nitrogen cannister is there for our nitrogren friends, with an emphasis on Vox. Even the Pioneer and Mc Delivery embraced the pain and they're about as small as it gets! At this point I believe that there are no shuttles with atmosphere that have not had this change. If you know otherwise feel free to pass it along so it can be fixed up! I imagine that was an oversight when it was pushed along, along with some of the small issues like unanchored canisters and so on. These things happen : ) The solution provided by @Jakumba was elegant and imposed a very modest footprint alteration I think! |
I do believe that for the smallest shuttles it should not be necessary since their form factor and size are meant for a very simple task and should not have the same accommodations than a regular sized shuttle but im not a maptainer, so if it is their decision that absolute all ships must have have no matter how big or small then sure. |
Hello! Atmos: Nice job adding a mixer.
|
Noted! I'll fix this pronto.
Seems sensible, perhaps replacing the single softsuit locker with a hardsuit locker might be the play? This would put on par with the Templar and Hospitaller, which both have hardsuit access.
I will look into how to do this, and set the PR to draft until that has been done.
I'm not a massive fan of external components either, space is very much a premium but I'll see if I can find a way to make it fit without compromising the shape of the shuttle.
I did toy with this, and adding a second seat next to the pilots. I'll give it a try and edit this with a screenshot for the sake of gathering additional feedback. Edit: I've now done the following:
|
About the PR
TODO:
Why / Balance
The paladin is dedicated attack ship for a crew of 2-3, but it has a single EVA suit locker. Now there are two hard suits for Heat Two situations. The atmos can was also not connected to the system (and could not, due to the ports placement by the window) so I've moved it down to make space.
How to test
Spawn in on NFSDO, buy a Paladin and take a walk around
Media
Requirements
Breaking changes
Shouldn't be any!
Changelog
🆑