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

update master #2360

Merged
merged 472 commits into from
Sep 28, 2024
Merged

update master #2360

merged 472 commits into from
Sep 28, 2024

Conversation

pcarruscag
Copy link
Member

No description provided.

sun5k and others added 30 commits May 2, 2024 15:07
Sarkar
Re
Sarkar Regression
Address sanitizer tests in the CI pipeline
Added mach number evaluation for the engine boundaries.

Decoupled the engine marker evaluation frequency from screen write frequency
Defining the option BC_EVAL_FREQ, for setting the evaluation frequency for the Engine and Actuator markers
Defining option BC_EVAL_FREQ, to set the evaluation frequency for Engine and Actuator Disk Markers
Test case for marker_engine_inflow
…estCases/engine/marker_engine_inflow/inv_channel_engine_inflow.cfg
Co-authored-by: Pedro Gomes <[email protected]>
joshkellyjak and others added 27 commits September 13, 2024 17:15
Signal handling - SIGTERM will stop calculation and force file saving
* extending CFL adapt to include species transport

* add underrelaxation factor for the last species that is not computed during simulations

* removing underrelaxation factor for species transport

* adding CFLSpeciesReduction factor
Corrections to SA implementation
* cleaning constructor fluid mixture

* formatting
Changing version number to 8.1.0
;
return newstring;
}

this->markers = new string[totalVals];

Check warning

Code scanning / CodeQL

Resource not released in destructor Warning

Resource markers is acquired by class COptionTurboPerformance but not released anywhere in this class.
@pcarruscag pcarruscag merged commit 0e1495c into master Sep 28, 2024
59 of 73 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.