-
Notifications
You must be signed in to change notification settings - Fork 12
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
Double-bubble cryogenic tanks #64
Conversation
updated docs Update bubble_geom.jl better comments
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #64 +/- ##
==========================================
+ Coverage 76.41% 78.34% +1.92%
==========================================
Files 69 72 +3
Lines 13169 13460 +291
==========================================
+ Hits 10063 10545 +482
+ Misses 3106 2915 -191 ☔ View full report in Codecov by Sentry. |
Closing as superseded by PR #80. |
This PR removes inconsistencies in the geometry of the cryogenic tanks in the fuselage. Some models assumed that the geometry was a double bubble while others only applied to circular sections. Now, the tank cross-section can be a double bubble in general, following the fuselage section.
This PR also introduces a new function
double_bubble_geom()
that can be used to calculate the geometric properties of a double bubble shape. This function could also be used infuseW
for the fuselage design.