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

Alerte á la Machinist rework: Adds additional corps (Zavod/Zeng), makes Machinist a civilian role #20395

Closed

Conversation

FearTheGabby
Copy link
Contributor

image

Summary: This PR redefines the Machinist as a "civilian" role, removing it from Operations for a particular purpose: Allowing more megacorporations to take the role without diluting the department-megacorp connection currently in place. There's been a lot of past discussion about what department Machinist is meant to fit into, and I decided to use a similar format to the one that the Corporate Reporter now goes by to hopefully satisfy those who don't feel it belongs in Operations as well as those who felt that some megacorporations who did robotics were left out of the fold.

The Machinist now gets the medical-research headset as well, both to communicate with Medical about patients who are synthetic or who have cybernetics, and to communicate with Research about R&D progress or other necessities. This does not make them part of either department, nor does it mean they have the relevant access.

I am happy to put up a feedback thread on the forums if needed, and to cooperate with maintainers as-needed if the PR is accepted in part or in full. Thank you.

@BotBOREALIS BotBOREALIS added the Sprites Adds new or changes existing sprites. label Jan 24, 2025
@FearTheGabby
Copy link
Contributor Author

!review

@github-actions github-actions bot added the 🗺️ Mapping - Horizon The PR touches the Horizon map files. label Jan 24, 2025
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@FlamingLily
Copy link
Contributor

Honestly? This might be the best option to include all four robotics/cybernetics companies. The job role is just too diverse to be limited to one department - cybernetics are firmly in the realm of medical and science, IPCs kind of flirt with engineering and science, while mechs are a bit more of a pure engineering-esque role, so it's kind of a mess all around, and I think this is the best way to do it

@N-Komaeda
Copy link

Something that would need clarification - do they have an immediate Head of Staff supervising them? Since they're under the Civilian branch, will that be the Executive Officer, or will it be fully like the reporter where they fall under the purview of their parent corporation and have strict privileges/rules on station? If the latter, I'd imagine there's also need for CCIA approval on this?

@FluffyGhoster
Copy link
Contributor

MD as this would need evaluation from both lore and CCIA perspective of how this would even function in such a scenario

@FluffyGhoster FluffyGhoster added the ⛔ Maintainer Discussion The PR will be discussed by the maintainers. label Jan 27, 2025
@NonQueueingMatt
Copy link
Contributor

Unfortunately, I cannot accept this PR. Reasons:

  • The machinist doesn't really fit in the civilian department -- by nature, it's a department that doesn't have any designated leader, much less any sort of real responsibilities, so it's thematically unfitting.
  • It being in the civilian department while having a medsci radio is very inconsistent.
  • We don't particularly want to bring back double radios because they cause a lot of confusion over who should listen to who (this is the reason why all non-head of staff roles have only one radio).
  • It is intentional that every department barring exceptions like liaison/reporter has two or three corps at most.

I do think machinist needs a lot of changes and the state of the role isn't satisfying to me, but this is the part where we have to realize that we're working with constraints placed on us by the setting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⛔ Maintainer Discussion The PR will be discussed by the maintainers. 🗺️ Mapping - Horizon The PR touches the Horizon map files. Review Required Sprites Adds new or changes existing sprites.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants