Skip to content
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

Feature/better translate test #36

Merged
merged 8 commits into from
Dec 29, 2023

Conversation

FlorianDeconinck
Copy link
Collaborator

Translate test: small improvements

  • Parametrize perturbation upon failure
  • Refactor error folder to be pwd based
  • Fix GPU translate unable to dump error nc
  • Fix mixmatch precision on translate test
  • Update README.md

Test fix:

  • Orchestrate YPPM for translate purposes

Misc:

  • Fix bad logger formatting on DaCeProgress

    Update dace to candidate 0.15.1
    Update gt4py to main branch as of 23/12/04 (d7cf10)
    Fixes to translate test to allow orchestrated one-rank regression test to pass
Minor adjustments to logging on orchestration build. Clean up.
Orchestrate YPPM (for translate purposes)
Fix mixmatch precision on translate test
Refactor error folder to be `pwd` based
Parametrize perturbation upon failure
Fix GPU translate unable to dump error nc
Update README.md
@FlorianDeconinck FlorianDeconinck merged commit a53b755 into geos/main Dec 29, 2023
2 checks passed
@FlorianDeconinck FlorianDeconinck deleted the feature/better_translate_test branch December 29, 2023 17:46
FlorianDeconinck pushed a commit that referenced this pull request Jan 30, 2024
* initial changes to read in ak bk

* read ak/bk

* add xfail

* remove input dir

* further changes to unit tests

* finish up test

* add history

* commit uncommited files

* fix test comment

* add input to top

* read in data

* read in netcdf file in eta mod

* remove txt file

* test

* modify test and fix generate.py

* remove emacs backup file

* driver tests pass

* fix helper.py

* fix fv3core tests

* fix physics test

* fix grid tests

* nullcommconfig

* cleanup input

* remove driver input

* remove top level input

* fix circular import problems

* modify eta_file readin for test_restart_serial

* comment out 91 test

* rm safety checks

* revert diagnostics.py

* restore driver.py

* revert initialization.py

* restore state.py

* restore analytic_init.py

* restore init_utils.py and analytic_init.py

* restore c_sw.py

* d2a2c_vect.py

* restore fv3core/stensils

* restore translate_fvdynamics

* restore physics/stencils

* restore stencils

* remove circular dependency

* use pytest parametrize

* cleanup generation.py

* fstrinngs

* add eta_file to MetricTerm init

* remove eta_file argument in new_from_metric_terms and geos_wrapper

* use pytest parametrize for the xfail tests

* use pytest parametrize for the xfail tests

* fix geos_wrapper and grid

* fix tests

* fstring

* add test comments

* fix util/HISTORY.md

* fix comments

* remove __init__.py from tests/main/grid

* add jupyter notebooks to generate eta files

* generate ak,bk,ptop on metricterm init

* fix tests

* exploit np.all in eta mod

* remove tests/main/grid/input

* update ci

* test

* remove input

* edit ci yaml

* remove push

---------

Co-authored-by: mlee03 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant