-
Notifications
You must be signed in to change notification settings - Fork 8
Update serialized data #1004
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
Open
jcanton
wants to merge
33
commits into
main
Choose a base branch
from
test_cartesian_coords
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Update serialized data #1004
Changes from 9 commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
46f7b43
add cartesian coordinates to serialbox
jcanton c021067
add cartesian coordinates to tests
jcanton 7b46f67
pre-commit
jcanton 2b1b0d4
update serialized data names
jcanton d384330
Merge branch 'main' into test_cartesian_coords
jcanton 35f3b31
Merge branch 'main' into test_cartesian_coords
jcanton d890878
grid.kind -> grid.shape and no more guessing
jcanton ad3b34e
unify names with icon-exclaim
jcanton bf31078
Add reserved ranks and versioning properties to Experiment class; imp…
jcanton 3830bc0
Fix data download paths
msimberg b3afef5
Fix formatting
msimberg 26e1542
shuffle functionality
jcanton 51dcb49
Small refactor of test data downloading
msimberg 6eff2d2
move to normal
jcanton cb50ad2
formatting
jcanton ebaab87
Make lockfile instead of suffix explicit
msimberg 3f242c5
Fewer assumptions
msimberg 8e33640
Formatting
msimberg 3303745
some update
jcanton 8266e7e
update extra_ranks
jcanton f7ad71b
script text -> content
jcanton eab98f6
move functionality
jcanton 0d4b0a5
undo cartesian related changes, now in #1013
jcanton 601412d
Merge fix-data-download from msimberg/icon4py
jcanton a4e682e
cleanup
jcanton 616524f
pre-commit
jcanton 04ce1d0
put this back
jcanton 4c508fc
cli Typer stuff
jcanton f583ae5
Merge branch 'main' into test_cartesian_coords
jcanton 98f64ac
flatten directory structure
jcanton 39827df
restrict search to nml block
jcanton 8b995ef
comm_size and tar toplevel
jcanton ac28470
run only these two for now
jcanton File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
We don't use the
REGIONALproperty anywhere?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.
nah, it's a leftover from who knows what, never used anywhere...
also grid root and level -> global_num_cells -> mean_cell_area can be removed, but will do the cleanup in a separate PR as that's purely cleaning