-
Notifications
You must be signed in to change notification settings - Fork 4
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
regrid-xy scripting to use pathlib for filename parsing #32
base: main
Are you sure you want to change the base?
Conversation
- remove undesired fms-user remap cache "feature" - remove some commented code - add automatic exit code checking for the fregrid system calls
The original problem was Tim noticing these files in my /xtmp directory:
Instead of just fixing the problem, I thought I'd "fix" other string filename parsing. But it took me a lot longer than it should have and I'm not sure it's greatly improved. |
did you check the tests in |
Let me applaud the turn-towards |
You got me- I must have missed something. With the main branch, 10 out of 12 pass, and with these changes, only 6 out of 12 pass. |
I take it back- I somehow was using an old branch. 10 / 12 still pass, with this update. |
OK so in light of fre-cli's add_regrid_xy PR do we want to merge this here now? we should start a new issue/branch of dev using the new |
I was hoping to be lucky enough to get the pipeline working for reviewing this. but Running the tests within the main branch on PPAN by hand, using a personally-built
As output i see:
Now, I checkout this branch and do the same thing!
and i see:
looking at the output, i see syntax errors. |
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.
see comments in main thread!
if you'd like, merge with the pipeline branch and you can get pylint feedback
resolves #31