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

Improved config for SABRE engines. #1884

Merged
merged 3 commits into from Sep 29, 2018
Merged

Conversation

dearmoon9
Copy link
Contributor

Better isp and minimal thrust.

@Bornholio
Copy link
Contributor

https://en.wikipedia.org/wiki/SABRE_(rocket_engine)
smoke up the butt says your ISP is good. IF specifications are derived from Wiki Sources TWR should be 14:1 with vacuum thrust of 2940kN

My comment on that is given the nozzle size i can't see that as possible unless thrust is far lower than specified. See RL-10's for ratio's that make sense (280:1 Nozzle = ISP 465). Or the chamber pressure is double or more an RS-25. RS-25 is ~69:1 ratio and only gets 452 ISP

@dearmoon9
Copy link
Contributor Author

14:1 is air-breathing TWR which has nothing to do with rocket mode (btw. current airbreathing TWR is too low and we need to increase that number as well).

Regarding nozzles I've found only this:
"Nozzle profiles similar to those of the Space Shuttle Main Engine are assumed"
https://www.nas.nasa.gov/assets/pdf/papers/Mehta_Unmeel_Skylon2015.pdf page 6

@dearmoon9
Copy link
Contributor Author

Some info about mass of the engine:
https://web.archive.org/web/20140127114808/http://www.reactionengines.co.uk/tech_docs/JBIS_v57_22-32.pdf page 32
Mass of the engine, inlet, bypass is equal to 7400 kg and after subtracting the mass of inlet (because it works in RO as a separate part) it will equal to 7000 kg.

@dearmoon9
Copy link
Contributor Author

dearmoon9 commented May 27, 2018

This page states that vacuum thrust would be only around 1800 kN
https://info.aiaa.org/tac/PEG/HSABPTC/Public%20Documents/Skylon%20Spaceplane.pdf - january 2012.

@raidernick raidernick added the Ready to Merge? Is this ready to merge? Needs feedback label Sep 29, 2018
Copy link
Contributor

@Bornholio Bornholio left a comment

Choose a reason for hiding this comment

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

Seems to work fine. (1.4.5 +RO.RP-0 install)

@raidernick raidernick merged commit 019d150 into KSP-RO:master Sep 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready to Merge? Is this ready to merge? Needs feedback
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants