Prep code for paper publishing #33
Merged
Conversation
This file contains 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
compared to their counterparts in the original repo
rather than library used to read data i.e. xarray -> ukcp18
even though empty as so far untested
related to FID/Inception scores that I don't use
this is a pytorch application so clearer to name it data
which are added when installing package
and bump mlde_utils now that it has debug logging in ClipT
to make room for multivar and inform using CPM from UKCP Local (not 18) also updated queue train helper to not require dataset now have them set to good versions in config by default
need to figure out why not working with loc spec channels but still better to have a smoke test that runs
and add a low data config
and move blue pebble helpers into separate dir
just cleaner and more consistent
particularly explaining the predict.py command
not needed anymore
Tidy deterministic (U-Net) code ready for paper publishing
for completeness to ensure separating different possible approaches to qmapping
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
So in particular make sure licensing and documentation is present and correct. Plus update some names (particularly of configs) to make them easier to find.