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
Make Lassen scripts cache rank-local
#881
Closed
matthiasdiener
closed
1 year ago
matthiasdiener
commented
1 year ago
Questions for the review
:
[x] Is the scope and purpose of the PR clear?
[x] The PR should have a description.
[x] The PR should have a guide if needed (e.g., an ordering).
[x] Is every top-level method and class documented? Are things that should be documented actually so?
[x] Is the interface understandable? (I.e. can someone figure out what stuff does?) Is it well-defined?
[x] Does the implementation do what the docstring claims?
[x] Is everything that is implemented covered by tests?
[x] Do you see any immediate risks or performance disadvantages with the design? Example: what do interface normals attach to?
Questions for the review: