-
Notifications
You must be signed in to change notification settings - Fork 26
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
Sync Path #462
Comments
Yeah that's a good idea. My only concern is that it would break existing usage of sync (if there is any), by syncing all output in a new subfolder in the I thought I had previously added in logic in the sync code to check if there was pre-existing metadata.yaml file with a matching UUID in the sync path to ensure it was the same experiment, but after looking at the code, I had not sorry.. |
I think its only an issue if users change payu version part way through an experiment? Or try to add to an existing experiment which was run on an older payu? The worst case is that an extra folder is created - it wouldn't be too hard to tidy up manually ... ? |
Yeah true, it would not be difficult to tidy up manually, and would only be an issue if changing payu versions during running an experiment. Just a couple more thoughts about implementation:
|
Should the sync path configure follow the style as short path? Where the experiment name is added to the path provided.
i.e. if:
should this create a folder
/g/data/$PROJECT/$USER/<<experiment_name+uuid>>
automatically to sync to?
This seems especially useful when using the
payu branch
from a point in another experiment and you don't want to override the existing results ?Current behavior is it will create the output folders directly in the path directory
The text was updated successfully, but these errors were encountered: