Skip to content
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

Remove the NCAR specific paths that don't allow things to work outside of Derecho #2133

Open
ekluzek opened this issue Sep 1, 2023 · 1 comment
Labels
priority: low Background task that doesn't need to be done right away. type: bug something is working incorrectly type: enhancement new capability or improved behavior of existing capability

Comments

@ekluzek
Copy link
Contributor

ekluzek commented Sep 1, 2023

Doing a "git grep glade" I see a lot of glade specific paths in our code. Some of this is acceptable, as these are contrib support codes only expected to work on NCAR machines. Some of it is also just setting up tools to work on specific machines (although some of those should be updated to use cime configure). Some of it's just documentation, so that's OK.

I see several testmod directories that have a dependence on absolute paths under /glade. The data here should be entered into inputdata, so these tests can be run elsewhere.

We should also add some error reporting so that a sensible error message is given about the problem when it's encountered.

Definition of done:

  • [ ]: Add error checking for tools/tests that require NCAR machines or NCAR specific paths
  • [ ]: Import test data that's just on /glade to CESM inputdata
  • [ ]: Remove the /glade specific paths for testmods and use data imported to inputdata
@ekluzek ekluzek added type: enhancement new capability or improved behavior of existing capability type: bug something is working incorrectly tag: next this issue should get some attention in the next week or two labels Sep 1, 2023
@ekluzek
Copy link
Contributor Author

ekluzek commented Sep 1, 2023

@billsacks billsacks added priority: low Background task that doesn't need to be done right away. and removed tag: next this issue should get some attention in the next week or two labels Sep 7, 2023
@ekluzek ekluzek changed the title Remove the NCAR specific paths that don't allow things to work outside of Cheyenne Remove the NCAR specific paths that don't allow things to work outside of Derecho Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: low Background task that doesn't need to be done right away. type: bug something is working incorrectly type: enhancement new capability or improved behavior of existing capability
Projects
None yet
Development

No branches or pull requests

2 participants