Turbulent inflow generation logic patch #1695
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 30 commits
June 25, 2024 08:07
…put + .rst documentation
…ariables. Renamed BPM and CPM tags to source and direct to keep neutrality. Need to add Ec, tp, and cell_data equations for direction add.
…ithin TimeIntegration/ERF_Advance.cpp for CPM cases. Need to implement options to swap back and forth on use side between BPM and CPM. Switch gear and start runtime optimization.
…ded outputs for PB_amplitude/PB_updateTime in seperate file when erf.v=1. Need to find a good way to generalize amplitude. Currently breaks code when amplitude is too large. CONTINUE with CPM debug, fast_rhs_fun debug with PB, and documentation with .rst file.
…n debug with PB, and documentation with .rst file
… boxes. Added in perturbation inflow .rst documents. CONTINUE with CPM debug, fast_rhs_fun debug with PB.
… boxes. Added in perturbation inflow .rst documents. CONTINUE with CPM debug, fast_rhs_fun debug with PB.
… format into file + verbose tags. Fixed parallel issue of boxes. Added in perturbation inflow .rst documents. CONTINUE with CPM debug, fast_rhs_fun debug with PB.
…ose tags. Fixed parallel issue of boxes. Added in perturbation inflow .rst documents. CONTINUE with CPM debug, fast_rhs_fun debug with PB.
…Currently blows up after 1st update interval.
… (2015). DOI links return Erorror 403, actual link returns Error 404.
added 19 commits
July 2, 2024 15:18
…hancial perturbation)
…hancial perturbation)
…pressible flow algorithm.
…nt. Random assignment happens at the calc_tpi_update() level
dustinma324
changed the title
…nerate dirichlet input file. Added more description into documentation.
dustinma324
force-pushed
the
perturbation_box_method
branch
from
2acb16e to
d9d7c86
Compare
added 2 commits
July 22, 2024 15:19
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This patch modifies the logic in two functions,
calc_tpi_update()andapply_tpi(). In this version, aMultiFabis created withinDataStructs/TurbPertStruct.Hto store the random perturbation amplitude per cell within the perturbation box. The randomization step is now pulled out ofapply_tpi(), which was incorrectly being called at every iteration, and placed withincalc_tpi_update().With this change, procedures such as calculating mean box velocity, perturbation amplitude, and assigning random perturbations are only triggered when the update interval is surpassed by the local elapsed time per box. This ensures that the randomization of perturbation assignment within the perturbation box occurs only once per update interval. The
Array4that stores the random perturbation is then used inapply_tpi()per iteration as a direct addition to the source term of the desired field.rhoThetafield.movie_incompressible_BPM.mp4