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 models for cryogenic fuel boiloff #29

Merged
merged 91 commits into from
Apr 16, 2024
Merged

Conversation

ngomezve
Copy link
Contributor

@ngomezve ngomezve commented Apr 2, 2024

This PR applies some minor improvements to the models for cryogenic fuels:

  • Boiloff fuel is only vented when it would result in an increase in tank pressure (if the volume of the boiloff gas at saturation is greater than the volume that is made available by liquid boiloff and fuel burn).
  • The mass of the ullage gas inside the tank is now accounted for.
  • The latent heat of vaporization of hydrogen and methane is now a function of tank pressure.
  • Added Mylar as a material option for the insulation, specifically for vapor barriers.
  • Font size in tank diagrams for docs has been increased.

ngomezve and others added 30 commits March 11, 2024 17:25
@codecov-commenter
Copy link

codecov-commenter commented Apr 2, 2024

Codecov Report

Attention: Patch coverage is 81.63265% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 74.76%. Comparing base (c224dd9) to head (5b1a55b).

Files Patch % Lines
src/IO/read_input.jl 0.00% 4 Missing ⚠️
src/structures/tankWthermal.jl 60.00% 4 Missing ⚠️
src/structures/tankWmech.jl 83.33% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #29      +/-   ##
==========================================
+ Coverage   74.73%   74.76%   +0.03%     
==========================================
  Files          69       69              
  Lines       13089    13111      +22     
==========================================
+ Hits         9782     9803      +21     
- Misses       3307     3308       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ngomezve ngomezve marked this pull request as draft April 2, 2024 15:48
@ngomezve
Copy link
Contributor Author

ngomezve commented Apr 2, 2024

I converted the PR back to draft to increase test coverage based on Codecov report.

@ngomezve ngomezve marked this pull request as ready for review April 2, 2024 21:23
@ngomezve ngomezve changed the base branch from main to joss April 4, 2024 13:46
@ngomezve ngomezve changed the base branch from joss to main April 4, 2024 13:46
@askprash askprash merged commit 9d8ef16 into MIT-LAE:main Apr 16, 2024
3 checks passed
askprash pushed a commit to askprash/TASOPT.jl that referenced this pull request May 3, 2024
* better saturation fuel properties

* better model for fuel venting

* removed println from mission()

* rewrote fuel calc in mission! for clarity

* added mylar as insulation opt for vapor layers

* simplified FFC calc in mission!

* fixed bug geom vars not updated

* increased font in tank diagrams

* better saturation fuel properties

* better model for fuel venting

* better density and enthalpy of vap for cryo fuels

* removed println from mission()

* rewrote fuel calc in mission! for clarity

* integrated new fuel_properties in read_input

* better latent heat of vaporization in tank models

* bug fix + new test for fuel_properties

* accounted for mass of ullage gas in tank

* updated tests after changing ullage calculation

* updated docs to reflect new ullage method
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