-
Notifications
You must be signed in to change notification settings - Fork 51
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
Simplify example configs #846
Conversation
It's very old and not really useful
@milenaveneziani, this follows up on the Cori updates from #845. I don't have time to test these examples on all centers, nor would I expect you to do so. But if you notice any obvious mistakes or situations where things are unclear, please let me know. |
This is a great cleanup @xylar. |
Okay, I can do that. I kept what I thought was the most recent run but it turns out to be sea-ice only (https://github.com/MPAS-Dev/MPAS-Analysis/blob/fdc260433c97393f85265ed3427009e4ce97ed43/configs/lanl/20170207.MPAS-SeaIce.QU60km_polar.wolf.cfg) and maybe not very useful. Honestly, the example config files for specific machines aren't too useful especially if they're really old like the LANL ones. The examples in the repo root are probably a better starting place. |
which one are you thinking as a good example for LANL machines? |
@milenaveneziani, sorry for the confusion on this. It's kind of a major change and it didn't get communicated to users very well. That's partly what I'm hoping to clarify with this clean-up. Now that
or call In each of the example job scripts, I have a comment like this:
That is the preferred approach at this point. You should no longer specify a |
oh right, now I remember! (that was also the reason why I couldn't understand that it wouldn't take the path that we gave it in the config file! (there was a small typo in the path set by |
Yes, indeed: |
I think this is fixed -- I just edited the machine files directly. I don't have any test analysis to run, though, on LANL IC. Let me know if you do and if it works. |
Thanks! Why don't you go ahead and merge this for now, and I'll try a test with Prajvala next time we meet in person (the way we went around it last time was to load e3sm-unified.1.5.0). I will let you know if there are still problems, but I don't foresee any. |
Thanks, @milenaveneziani! |
This merge simplifies the example config files and jobs scripts for other HPC centers following the clean-up in #845