issues
search
illinois-ceesd
/
mirgecom
MIRGE-Com is the workhorse simulation application for the Center for Exascale-Enabled Scramjet Design at the University of Illinois.
Other
12
stars
19
forks
source link
Update Riemann and Linear BCs
#1002
Closed
tulioricci
closed
9 months ago
tulioricci
commented
9 months ago
Update
species_mass_fractions
for non-mixture cases in RiemannBC.
Allow either to prescribe or to extrapolate species in LinearOutflowBC; also, do not enforce the gradients to be zeros (over-specified).
Extended LinearInflowBC and LinearOutflowBC to 1D
Questions for the review
:
[ ] Is the scope and purpose of the PR clear?
[ ] The PR should have a description.
[ ] The PR should have a guide if needed (e.g., an ordering).
[ ] Is every top-level method and class documented? Are things that should be documented actually so?
[ ] Is the interface understandable? (I.e. can someone figure out what stuff does?) Is it well-defined?
[ ] Does the implementation do what the docstring claims?
[ ] Is everything that is implemented covered by tests?
[ ] Do you see any immediate risks or performance disadvantages with the design? Example: what do interface normals attach to?
species_mass_fractions
for non-mixture cases in RiemannBC.Questions for the review: