Open mukul1992 opened 1 month ago
Most of my comments right now are just noting the obvious things to clean up before actually merging. Thank you for taking this task on!!
Speaking of merging, it looks like there are some complicated merge conflicts between this branch and main. It may be helpful to rebase your branch to upstream/main to remove some of those conflicts and get the unrelated parts of this branch up-to-date. Removing the merge conflicts will also allow the CI tests to run.
@marchdf Yes, I will certainly clean up all the debug statements and incorporate other changes you recommended.
@mbkuhn Yes, I will do the rebase over the next week along with cleaning things up.
Thanks for the early feedback!
PS. I have been able to rebase this code with the latest changes in the Exawind/amr-wind
repo. I have also updated the amrex submod in the index (let me know if I did it incorrectly). And have updated the BCType from user_1
to direction_dependent
in the code.
And finally, here's the draft PR for AMReX-Hydro that includes the outflow-correction for enforcing solvability.
Summary
Enabling the
mass_inflow_outflow
(MIO) BC which allows for both inflow and outflow cells in a boundary.Pull request type
Please check the type of change introduced:
Checklist
The following is included:
This PR was tested by running:
Additional background
This functionality would be required for coupling with ERF and was started after discussions with @gdeskos and @mchurchf, along with @asalmgren, as it was a desired functionality within AMR-Wind too. Additionally, @hgopalan has expressed interest, and @marchdf has helped along the way.
Description
The changes required can be organized into six categories:
MassInflowOutflowBC.cpp
ext_dir
andfoextrap
behavior respectively.enforceSolvability
routine in AMReX-Hydro before the mac-projection, which adjusts the outflux to match with influx (will link to the corresponding PR in Hydro).input_inout
.