-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add physics savepoint test #5
Conversation
`gtc:gt:cpu_ifirst` threshold update
… add-physics-savepoint-test
… add-physics-savepoint-test
Reset the grid only if we have multiple ranks Use numpy API to calculate worst error Re-use code for compare_scalar
…Modeling/fv3gfs-integration into add-physics-savepoint-test
…integration into add-physics-savepoint-test
…Modeling/fv3gfs-integration into add-physics-savepoint-test
"va": {}, | ||
} | ||
|
||
def collect_input_data(self, serializer, savepoint): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All functions except for compute_parallel
can be removed once Grid initialization is ready
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is an impressive amount of work. Will the overlap with fv3core
, overrides, translate classes, etc., be combined as we move toward a mono repo?
|
||
|
||
# get MPI environment | ||
sys.path.append("/usr/local/serialbox/python") # noqa: E402 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this a reliable location for serialbox?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It isn't, but it is how fv3core
has it for now. I think the best solution is to be able to pip install serialbox, otherwise we may want to move to an environment variable for this.
GT4PY_VERSION.txt
Outdated
v35 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just curious if there are problems in v36
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has been tested with v36 for GTC backends, it should be updated
Thanks Eddie for reviewing this! This infrastructure is indeed shared across dycore and physics. Our next step will be to move |
Moving/duplicating some of the infrastructure in
fv3core
tests up to the integration directory. In this PR, we include savepoint tests for physics only.