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

Fix marker engine inflow #2278

Merged
merged 23 commits into from
Sep 5, 2024

Conversation

BerkeCan97
Copy link
Contributor

@BerkeCan97 BerkeCan97 commented May 11, 2024

Proposed Changes

  • Added the missing config->SetInflow_Mach(iMarker_Inlet, Inlet_Mach_Total[iMarker_Inlet]) to CEulerSolver::GetPower_Properties
  • Added a new option,BC_EVAL_FREQ for setting the evaluation frequency for the Actuator Disk and Engine Markers, in order to control the stability of the marker.

Related Work

Fix for issue #2274 .

PR Checklist

Put an X by all that apply. You can fill this out after submitting the PR. If you have any questions, don't hesitate to ask! We want to help. These are a guide for you to know what the reviewers will be looking for in your contribution.

  • I am submitting my contribution to the develop branch.
  • My contribution generates no new compiler warnings (try with --warnlevel=3 when using meson).
  • My contribution is commented and consistent with SU2 style (https://su2code.github.io/docs_v7/Style-Guide/).
  • I used the pre-commit hook to prevent dirty commits and used pre-commit run --all to format old commits.
  • I have added a test case that demonstrates my contribution, if necessary.
  • I have updated appropriate documentation (Tutorials, Docs Page, config_template.cpp), if necessary.

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
config_template.cfg Outdated Show resolved Hide resolved
Common/include/CConfig.hpp Outdated Show resolved Hide resolved
Common/include/CConfig.hpp Outdated Show resolved Hide resolved
BerkeCan97 and others added 5 commits May 13, 2024 08:51
Co-authored-by: Pedro Gomes <[email protected]>
Changed the mesh to an existing mesh in the repository
Changing the mesh file to an existing mesh
config_template.cfg Outdated Show resolved Hide resolved
Common/src/CConfig.cpp Outdated Show resolved Hide resolved
BerkeCan97 and others added 2 commits May 28, 2024 13:10
@BerkeCan97
Copy link
Contributor Author

Hello

I am sorry if it is obvious but is there way to tell where the code style checks fails on the files? Looking at the the details section , only thing I can understand is that it failed to trim some trailing white spaces.

@pcarruscag
Copy link
Member

If you install the precommit hook it will take care of formatting for you https://su2code.github.io/docs_v7/Style-Guide/

@bigfooted
Copy link
Contributor

@BerkeCan97 can you run pre-commit on the files? We can then continue merging this.

@BerkeCan97
Copy link
Contributor Author

@BerkeCan97 can you run pre-commit on the files? We can then continue merging this.

Done, sorry it took so long.

Common/src/CConfig.cpp Outdated Show resolved Hide resolved
Copy link
Member

Choose a reason for hiding this comment

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

If this file is not used in one of the *_regression.py files we should remove it.

@pcarruscag pcarruscag merged commit 13763d8 into su2code:develop Sep 5, 2024
2 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.

4 participants