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

Investigator (Update) #1671

Conversation

ErhardSteinhauer
Copy link
Contributor

About the PR

  • Removed bar and microwave.
  • Removed salvage equipment.
  • Created separate engine room.
  • Reworked atmos.
  • Added guidebook entry with map.

Why / Balance

Part of the ongoing push for limiting shuttles to single purpose. Investigator enjoyers had it too good for far too long.

How to test

Buy NR Investigator and despair.

Media

2024-7-14_11 06 20
2024-7-14_11 06 04
image

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

Breaking changes

Investigator powergaming

Changelog
🆑 erhardsteinhauer

  • tweak: Updated NR Investigator.

@github-actions github-actions bot added Docs Improvements or additions to documentation Map-Shuttle Map - Shuttle No C# Sprites FTL labels Jul 14, 2024
Copy link
Contributor

github-actions bot commented Jul 14, 2024

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

Resources/Textures/_NF/Guidebook/shuttle_maps/128x96.rsi

State Old New Status
investigator Added

Edit: diff updated after ac54c9e

@ErhardSteinhauer
Copy link
Contributor Author

Ship owner: @UbaserB
(Just tagging you to make sure you're aware there's suggested changes for you ship.)

@dvir001
Copy link
Contributor

dvir001 commented Jul 14, 2024

Might want to keep microwave just for the rads reactions

@ErhardSteinhauer
Copy link
Contributor Author

Might want to keep microwave just for the rads reactions

They can research service branch and print microwave.
Or buy refined bananium.

@ErhardSteinhauer
Copy link
Contributor Author

Or spray it

@dvir001
Copy link
Contributor

dvir001 commented Jul 14, 2024

Might want to keep microwave just for the rads reactions

They can research service branch and print microwave. Or buy refined bananium.

Rads removed from it upstream and we probably just going to keep that change

@ErhardSteinhauer
Copy link
Contributor Author

Might want to keep microwave just for the rads reactions

They can research service branch and print microwave. Or buy refined bananium.

Rads removed from it upstream and we probably just going to keep that change

Hm, riiiight. Still, research option is there.

@UbaserB
Copy link
Contributor

UbaserB commented Jul 14, 2024

I don't like the tabletop computers, i prefer normal ones. Also, there's a shutter over the door in the control room. Other than that, lgtm

@ErhardSteinhauer
Copy link
Contributor Author

I don't like the tabletop computers, i prefer normal ones. Also, there's a shutter over the door in the control room. Other than that, lgtm

Shutter over the door is intentional

@dvir001
Copy link
Contributor

dvir001 commented Jul 14, 2024

Might want to keep microwave just for the rads reactions

They can research service branch and print microwave. Or buy refined bananium.

Rads removed from it upstream and we probably just going to keep that change

Hm, riiiight. Still, research option is there.

Fair

@UbaserB
Copy link
Contributor

UbaserB commented Jul 14, 2024

I don't like the tabletop computers, i prefer normal ones. Also, there's a shutter over the door in the control room. Other than that, lgtm

Shutter over the door is intentional

I can see the idea you're going for, but generally please just keep the shutters external (so like, between the seating area and the control room, no shutters)

@ErhardSteinhauer
Copy link
Contributor Author

but generally please just keep the shutters external

I disagree with this.
However, I don't have too strong feelings about it and I have editor opened right now, so I'll remove it.

@ErhardSteinhauer
Copy link
Contributor Author

2024-7-14_11 30 10

@ThatOneGoblin25
Copy link
Contributor

Hi do you reckon you could semi-revert the artifact chamber from the old version to have a gap between the ship and the chamber?

20240326211859!Investigator

I recently got told that the firelocks can be a bit of a death trap and I only made that change because the artifact analyse console had a smaller range.

@ErhardSteinhauer
Copy link
Contributor Author

I love me a good death trap. And I like the firelock setup there. But I'll think about it more.

Copy link
Contributor

@whatston3 whatston3 left a comment

Choose a reason for hiding this comment

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

Changes seem sane, ship looks good. Requested a small guidebook rewrite, otherwise, happy to approve.

@Dysh3s
Copy link

Dysh3s commented Jul 15, 2024

No bar = no ATM. ATMs are placed at ships with bar\kitchens, iirc. Or so i heard.

@ErhardSteinhauer
Copy link
Contributor Author

No bar = no ATM. ATMs are placed at ships with bar\kitchens, iirc. Or so i heard.

Not necessarily. In any case, the ATM is there to speed up the transactions between scientist and someone buying tech from them. (In rare case a scientist even cares to read the chat and sell something at all).

@Dysh3s
Copy link

Dysh3s commented Jul 15, 2024

No bar = no ATM. ATMs are placed at ships with bar\kitchens, iirc. Or so i heard.

Not necessarily. In any case, the ATM is there to speed up the transactions between scientist and someone buying tech from them. (In rare case a scientist even cares to read the chat and sell something at all).

Yeah, that's exactly why i asked for ATM for all RND ships at some point and was answered that only ships with bars\kitchens had ATMs.

@Tych0theSynth
Copy link
Contributor

No bar = no ATM. ATMs are placed at ships with bar\kitchens, iirc. Or so i heard.

Not necessarily. In any case, the ATM is there to speed up the transactions between scientist and someone buying tech from them. (In rare case a scientist even cares to read the chat and sell something at all).

Yeah, that's exactly why i asked for ATM for all RND ships at some point and was answered that only ships with bars\kitchens had ATMs.

Not sure how long ago that was, but no questions were raised about the Lyrae having an ATM when I PR'ed that. Doubt it's an issue any more, and it makes perfect sense for sci ships to have them.

@MagnusCrowe MagnusCrowe merged commit 314b7e5 into new-frontiers-14:master Jul 22, 2024
14 checks passed
FrontierATC added a commit that referenced this pull request Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Docs Improvements or additions to documentation FTL Map-Shuttle Map - Shuttle No C# Sprites
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants