Open lee-hyeoksu opened 1 month ago
Attention: Patch coverage is 83.01887%
with 9 lines
in your changes missing coverage. Please review.
Project coverage is 57.95%. Comparing base (
2810b09
) to head (5e41432
).
Files | Patch % | Lines |
---|---|---|
src/simulation/m_bubbles.fpp | 85.10% | 3 Missing and 4 partials :warning: |
src/simulation/m_time_steppers.fpp | 60.00% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description
This PR includes some updates on adaptive time stepping for sub-grid bubbles to improve the accuracy of the scheme. Also, I tried to make the subroutine
s_compute_bubble_source
be more concise by modularizing some parts into a few separate subroutines.This PR basically does not fix any bug nor introduce new features.
Type of change
Please delete options that are not relevant.
Scope
If you cannot check the above box, please split your PR into multiple PRs that each have a common goal.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Test Configuration:
Checklist
docs/
)examples/
that demonstrate my new feature performing as expected. They run to completion and demonstrate "interesting physics"./mfc.sh format
before committing my codeIf your code changes any code source files (anything in
src/simulation
)To make sure the code is performing as expected on GPU devices, I have:
nvtx
ranges so that they can be identified in profiles./mfc.sh run XXXX --gpu -t simulation --nsys
, and have attached the output file (.nsys-rep
) and plain text results to this PR./mfc.sh run XXXX --gpu -t simulation --omniperf
, and have attached the output file and plain text results to this PR.