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

Spectre (Update) #1072

Closed
wants to merge 48 commits into from
Closed

Spectre (Update) #1072

wants to merge 48 commits into from

Conversation

ThatOneGoblin25
Copy link
Contributor

@ThatOneGoblin25 ThatOneGoblin25 commented Mar 2, 2024

About the PR

  • Reworked all rooms
  • Wiring rework
  • Atmos rework
  • Decal rework
  • Added atmos piping to artifact chambers
  • Added robotics
  • Added plasma, carbon and empty canisters
  • Added a power drill as a RD item
  • Added a science techfab
  • Added a television to bar
  • Added Donut Vend
  • Added a Genedrobe with a secret wall below it
  • Replaced most consoles with tabletop variants
  • Removed blast doors over artifact chamber entrance
  • Removed medical doctor's wall locker
  • Removed some of the artifact containers
  • Trimmed the amount of lights
  • Tweaked bar, booze-o-mat instead of soda dispenser
  • Tweaked price, from 195000 to 1850000
  • Renamed from NSS Spectre to NR Spectre
  • New description.

Why / Balance

Updating to new mapping standards, also only large ship to be missing a mothership console.

Media

Screenshot 2024-03-23 003631
Screenshot 2024-03-23 003618
Screenshot 2024-03-23 003554
Screenshot 2024-03-23 003533

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
🆑

  • Tweak: Updated the Spectre to current mapping guidelines with additional features.

Copy link
Contributor

github-actions bot commented Mar 2, 2024

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

Resources/Textures/_NF/Structures/Machines/computers.rsi

State Old New Status
aiupload_key Added
computer_science Added
shipyard_science Added

Edit: diff updated after ee193f4

@ThatOneGoblin25
Copy link
Contributor Author

@Wolfhauler Letting you know I've proposed some changes to your ship.

@ThatOneGoblin25
Copy link
Contributor Author

Map checker failed because of the telecomms having Cargo, Engineering, Science and Service encryption keys.

@Wolfhauler
Copy link
Contributor

o7

@Wolfhauler
Copy link
Contributor

crematorium was denied last time by the by

@MagnusCrowe
Copy link
Contributor

I don't understand why a research ship would have a mothership console that allows you to get Pioneer, Prospector, Chisel, Bocadillo, PTS, Construct. It makes no sense to me.

Why the Smile AI?

@ThatOneGoblin25
Copy link
Contributor Author

ThatOneGoblin25 commented Mar 10, 2024

I don't understand why a research ship would have a mothership console that allows you to get Pioneer, Prospector, Chisel, Bocadillo, PTS, Construct. It makes no sense to me.

Drawing from its description: "A large research mothership designed to be flown next to a small fleet of other ships including salvage and food services." It should be able to provide these ships conveniently to the crew on board. The Crescent mothership console provides all the ships I listed and more.

There wasn't much point I saw in adding additional sci ships like the Sparrow or Investigator, since they do the same job as the main ship and we want to keep activity on there. I did have a vague idea to make a unique ship, a solar-powered shuttle that only had an artifact scanner and analysis console for this console but I realised it would detract from what the Spectre could offer.

As for the Smile AI, it's something unique that draws parallels to sci labs on stations. The largest sci-ship on the market should draw similarities to such sci labs. But it can go as the ghost role isn't whitelisted, you can't DO anything other than talk or suicide but if you believe it unnecessary I can remove it.

@MagnusCrowe
Copy link
Contributor

Nah, keep Smile. Makes more sense for it to be on the Spectre than the Wasp anyway

@dvir001
Copy link
Contributor

dvir001 commented Mar 12, 2024

Nah, keep Smile. Makes more sense for it to be on the Spectre than the Wasp anyway

Fucker worth 3k dead

@MagnusCrowe
Copy link
Contributor

Nah, keep Smile. Makes more sense for it to be on the Spectre than the Wasp anyway

Fucker worth 3k dead

I feel like that is a separate issue that should be addressed...why is it worth that much?

@ThatOneGoblin25
Copy link
Contributor Author

ThatOneGoblin25 commented Mar 13, 2024

I really feel like the Spectre deserves a mothership console still. Prehaps nerfed slightly in its options. It's really a sci base away from station and a substitute for Anom Labs with the Anomaly generator. As for docks, even with only two I think it can be managed by any crew, it is a tiny obstacle that can be overcome.

It needs a the bare minimum, the Pioneer on a mothership console.
The Spectre needs to work in conjunction with a salvage ship as its pure focus on science requires the gathering of ore and artifacts, even the anom gen can't be run roundstart as it needs bananium. It seems kinda poor taste to not provide any options for salvage on a sci ship, with the only other civilian science ship that does this being the Sparrow.

Also map checker fails for the encryption keys in the telecomms machine, it's something the original creator added, should I remove it @dvir001 ?

@github-actions github-actions bot removed the Sprites label Mar 22, 2024
@MagnusCrowe
Copy link
Contributor

I cannot actually build the code in this PR so I can't review it at this time. Does it build for you?

@ThatOneGoblin25
Copy link
Contributor Author

Added the NX Ghost shuttle for the Spectre Mothership Console.
Screenshot 2024-04-02 161034

@ThatOneGoblin25
Copy link
Contributor Author

I am like 90% sure i screwed up my own fork and if I reopened this on a new branch this wouldn't happen :P

@ThatOneGoblin25
Copy link
Contributor Author

Let's try opening a new branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Map-Shuttle Map - Shuttle No C# S: Awaiting Changes This PR has changes that need to be made before merging Sprites
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants