Skip to content

Roboticist#801

Merged
VMSolidus merged 6 commits intoSimple-Station:masterfrom
Mnemotechnician:feat/robostiscist
Sep 6, 2024
Merged

Roboticist#801
VMSolidus merged 6 commits intoSimple-Station:masterfrom
Mnemotechnician:feat/robostiscist

Conversation

@Mnemotechnician
Copy link
Copy Markdown
Contributor

@Mnemotechnician Mnemotechnician commented Aug 25, 2024

Description

By a (somewhat) popular request from floofstation, this adds the roboticist job (each existing map gets 1 job slot, since robotics on each of them is just one tiny room that's more similar to a broom closet than a subdepartament).

The job is purely cosmetical and does not offer any new accesses since that would require changing a lot of maps and would probably make being a scientist in the absense of a roboticist miserable. If needed, roboticists can always build blast doors on the entrance to robotics, as someone has done before in my experience.

This also adds the robotics PDA, robotics ID card, robotics satchel, backpack, and duffel (duffel icon taken from paradise station, rest of the sprites is respective science bags recolored by me to match the palette of the icon). This also increases the capacity of mystagogue's science PDA box to incorporate the new PDA type.

Media

image

image

image

image

image


Changelog

🆑

  • add: A new Roboticist job has been added. The job does not offer any new accesses, but lets you distinguish yourself from scientists. Existing maps will need to be updated to support the new job.
  • add: Added new PDA, ID, and bag variants for roboticists. The mystagogue will now also find a roboticist PDA in their office.

@github-actions
Copy link
Copy Markdown
Contributor

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

@github-actions github-actions bot added Status: Merge Conflict FIX YOUR PR AAAGH Changes: Localization Changes any ftl files Changes: Map Changes any yml file in the Maps directories Changes: Sprite Changes any png or json in an RSI labels Aug 25, 2024
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Aug 25, 2024

RSI Diff Bot; head commit 8c41633 merging into ec11eed
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Clothing/Back/Backpacks/robotics.rsi

State Old New Status
equipped-BACKPACK Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/Clothing/Back/Duffels/robotics.rsi

State Old New Status
equipped-BACKPACK Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/Clothing/Back/Satchels/robotics.rsi

State Old New Status
equipped-BACKPACK Added
icon Added
inhand-left Added
inhand-right Added

Edit: diff updated after 8c41633

@github-actions github-actions bot removed the Status: Merge Conflict FIX YOUR PR AAAGH label Aug 25, 2024
Copy link
Copy Markdown
Contributor

@DangerRevolution DangerRevolution left a comment

Choose a reason for hiding this comment

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

make a spawner for robotocist

@Mnemotechnician
Copy link
Copy Markdown
Contributor Author

I'll see if I can just replace-by-regex those spawnpoints in later

@github-actions github-actions bot added the Status: Needs Review Someone please review this label Aug 26, 2024
@DangerRevolution DangerRevolution added Priority: 4-Low Should be resolved at some point Size: 3-Medium For medium issues/PRs Type: Feature Creation of or significant changes to a feature and removed Status: Needs Review Someone please review this labels Aug 26, 2024
Copy link
Copy Markdown
Member

@VMSolidus VMSolidus left a comment

Choose a reason for hiding this comment

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

REmove

This reverts commit a28c214.
@github-actions github-actions bot added Changes: YML Changes any yml files and removed Changes: Map Changes any yml file in the Maps directories labels Aug 29, 2024
@github-actions github-actions bot added the Status: Needs Review Someone please review this label Aug 29, 2024
@Mnemotechnician
Copy link
Copy Markdown
Contributor Author

ok

@Elijahrane
Copy link
Copy Markdown
Contributor

Should be part of engineering

@OldDanceJacket
Copy link
Copy Markdown
Contributor

Should be part of engineering

If we're having golems and robots, then you should move robots to engineering, Epi is really big, I agree with this actually.

@Mnemotechnician
Copy link
Copy Markdown
Contributor Author

Should be part of engineering

Absolutely no.

@DEATHB4DEFEAT
Copy link
Copy Markdown
Member

Should be part of engineering

@ShadesMars
Copy link
Copy Markdown
Contributor

We need normal Science back and just EPI be like Robotist, a side of Science and not a compleat replacement.

@OldDanceJacket
Copy link
Copy Markdown
Contributor

OldDanceJacket commented Sep 4, 2024

Not going to let this sit because barely any discussion was had on the topic, it has no mechanics at all to it, so it can be merged instead of pointlessly sitting here dead. However the issue should still be discussed and will come back up once Roboticist actually has mechanics to it.

We need normal Science back and just EPI be like Robotist, a side of Science and not a compleat replacement.

The current direction of EE is psionics, so Epi has replaced Science and anything in the future should take that in mind. This is the direction we're currently going in, if you feel like this shouldn't be the case then make a discussion about it on discord. As it sits one of the big contributors(Solid) has been making Psionic focused PRs, and a number of PRs from other users have been returning missing content from Nyano that was lost in the DeltaV rebase.

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

Labels

Changes: Localization Changes any ftl files Changes: Sprite Changes any png or json in an RSI Changes: YML Changes any yml files Priority: 4-Low Should be resolved at some point Size: 3-Medium For medium issues/PRs Status: Needs Review Someone please review this Type: Feature Creation of or significant changes to a feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants