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
Improvements on filemovements #1162
Conversation
… no files are copied to the intermediate folders
This week I will be back. Hopefully be Wednesday or so, I get to give this a test. |
I consider this finished, so feel free to give it a try. I will work on the parallelization on another PR, and that would take longer. PD: Wednesday is bank holiday ;) |
Hi @mjy1011, I'll be looking into this today. The problem is that the preprocessing scripts expect files in the intermediate folders. |
Hi @mjy1011, I've got done some progress today with the issue, but there are some weird errors still. I think I am close, but I'll have to continue tomorrow. |
That's nice. Thank you for the update. |
12edb36
to
339da3d
Compare
…writing files in the general experiment folder when copying directly from work
Hi @mjy1011, That was a real tough one... but I think it's working now. Can you please try? @JanStreffing, can you try if the results are bit identical with |
Hi ~~ |
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.
Looks good!
Only a few minor changes (and one typo 😉)
Co-authored-by: Paul Gierz <pgierz@awi.de>
Co-authored-by: Paul Gierz <pgierz@awi.de>
Co-authored-by: Paul Gierz <pgierz@awi.de>
Co-authored-by: Paul Gierz <pgierz@awi.de>
Thanks for the review! |
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.
I agree with the error approach!
Just confirmed it is working right! And most of all, saving more than 2 hours, amazing. Thank you very much! |
#approve-changes |
You are welcome, and thanks for your patience with this issue :) |
Co-authored-by: Paul Gierz <pgierz@awi.de>
#bump |
This PR aims to:
config
,input
, andforcing
. The defaults can be overridden bygeneral.intermediate_movement = [list of filetypes that go to the intermediate folders]
This is to reduce the copying times of @tsemmler05, @a270067 and @mjy1011.
@JanStreffing and @mjy1011, this branch is not in it's final stage, but it would be a great help if you can check that is working for an actual simulation. I've tested already a bit but not in depth.