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

ThermoData branch #294

Merged
merged 57 commits into from
Apr 15, 2024
Merged

ThermoData branch #294

merged 57 commits into from
Apr 15, 2024

Conversation

GBenedett
Copy link
Collaborator

@GBenedett GBenedett commented Feb 5, 2024

Add-on module to include engine boundary conditions for thermodynamic calculations

  • Reorder the module
  • Improve old test / add tests
  • RANS SU2 Configuration File
  • Add ReadMe
  • Add pycycle to requirement

@GBenedett GBenedett self-assigned this Feb 5, 2024
@GBenedett GBenedett added SU2 Related to SU2 Python Related to Python Results Related to results generated by CEASIOMpy labels Feb 5, 2024
Copy link

codecov bot commented Feb 5, 2024

Codecov Report

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

Project coverage is 63.22%. Comparing base (b48fda8) to head (2611e15).

Files Patch % Lines
ceasiompy/SU2Run/func/su2config_rans.py 7.49% 247 Missing ⚠️
ceasiompy/SU2Run/func/su2config.py 20.00% 16 Missing ⚠️
ceasiompy/ThermoData/func/turbojet_func.py 91.30% 10 Missing ⚠️
ceasiompy/ThermoData/func/turbofan_func.py 97.76% 4 Missing ⚠️
ceasiompy/SU2Run/func/su2utils.py 25.00% 3 Missing ⚠️
ceasiompy/ThermoData/tests/test_thermodata.py 95.00% 3 Missing ⚠️
ceasiompy/SU2Run/su2run.py 75.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #294      +/-   ##
==========================================
- Coverage   63.76%   63.22%   -0.55%     
==========================================
  Files          84       88       +4     
  Lines        8150     8804     +654     
==========================================
+ Hits         5197     5566     +369     
- Misses       2953     3238     +285     
Flag Coverage Δ
integrationtests 44.09% <11.55%> (-1.52%) ⬇️
unittests 51.05% <52.20%> (+0.11%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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


<br />

`ThermoData` is a module to provide the outlet conditions of a given engine. It can calculate different operating conditions and save the results in a text file. This module is derived starting from the OpenSource code [pyCycle](https://github.com/OpenMDAO/pycycle) developed by Eric S. Hendricks and Justin S. Gray. It can perform calculations on both turbojet and turbofan engines, with the possibility to customize the parameters. The results are automatically written inside the config file of the module `SU2Run` to be able to perform the calculations.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

which parameters?

configuration file

`ThermoData` is a module to provide the outlet conditions of a given engine. It can calculate different operating conditions and save the results in a text file. This module is derived starting from the OpenSource code [pyCycle](https://github.com/OpenMDAO/pycycle) developed by Eric S. Hendricks and Justin S. Gray. It can perform calculations on both turbojet and turbofan engines, with the possibility to customize the parameters. The results are automatically written inside the config file of the module `SU2Run` to be able to perform the calculations.

## Inputs
`ThermoData` can be run on is own by giving the altitude the Mach number and the Net force. Otherwise it can take as an input the values from `CPACS2GMSH` module.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

altitude and Mach are inside aeromap, maybe change the sentence

`ThermoData` compute the values obtained at the engine outlet giving a "EngineBC.dat" file as an output. if the workflow continues with the `SU2Run` run module the results are added to the config file to perform the simulation.

## Outputs
`ThermoData` output is the "EngineBC.dat" file with stored inside: T_tot_out, V_stat_out, MN_out, P_tot_out, massflow_stat_out, T_stat_out, P_stat_out for the chosen engine configuration.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

add complete names, i.e. T_tot_out= Nozzle outlet temperature

Copy link
Collaborator Author

@GBenedett GBenedett left a comment

Choose a reason for hiding this comment

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

Ok

@GBenedett GBenedett merged commit f1ee8de into main Apr 15, 2024
4 checks passed
@GBenedett GBenedett deleted the add_ThermoData branch April 15, 2024 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Python Related to Python Results Related to results generated by CEASIOMpy SU2 Related to SU2
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants