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

Add RD-0108, improve RD-0110 configs #2909

Merged
merged 2 commits into from
Aug 31, 2023
Merged

Add RD-0108, improve RD-0110 configs #2909

merged 2 commits into from
Aug 31, 2023

Conversation

Capkirk123
Copy link
Member

Add RD-0108 config, a more reliable, man-rated version of the RD-0107. Also adjust RD-0106 config to be the same as the RD-0107, give 107% throttle to RD-0110, and decrease mass of all RD-0110 variants

@github-actions
Copy link

github-actions bot commented Aug 30, 2023

Download the artifacts for this pull request:

@NathanKell
Copy link
Member

Yay!! :)

One thing--if the 0106 is the same as the 0107, why do both configs exist rather than (I think we do this elsewhere?) combining them?

Since the RD-0106 is functionally identical to the 0107, just disable it.
@Capkirk123
Copy link
Member Author

I don't see why not

@lpgagnon
Copy link
Contributor

rp-1 was making 0106 unlock later but be cheaper: https://github.com/KSP-RO/RP-1/blob/ceec61b4557351f9e0adeaefc16dfae56e277968/Source/Tech%20Tree/Parts%20Browser/data/Engine_Config.json#L9836

can't speak to whether that was just nonsense (or to what the engine recosting has to say about it)

@Capkirk123
Copy link
Member Author

RD-0106 did fly later than the RD-0107. But since they were identical other than mounting, I don't really see any point to keeping it around.

@NathanKell NathanKell merged commit d4af0a4 into master Aug 31, 2023
3 checks passed
@NathanKell NathanKell deleted the RD0108 branch August 31, 2023 05:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants