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

Add ability to xmlchange to modify NUOPC streams xml files (or add a new tool to do so) #3950

Closed
ekluzek opened this issue Apr 30, 2021 · 5 comments
Labels
Responsibility: CESM Responsibility to manage and accomplish this issue is through CESM tp: user interface ty: enhancement

Comments

@ekluzek
Copy link
Contributor

ekluzek commented Apr 30, 2021

I'd like the ability for streams files such as created in CDEPS to be editable using xmlchange. Or add a new tool that would be specific for the streams files (called something like streamchange). @jedwards4b thought it should be possible to use xmlchange though using the "--subgroup" option group-name to set the stream name the change would apply to.

I think this is an important capability in order to allow users to easily change specific settings in the streams files without having to touch others. This is especially important when a set of changes for a case needs to be applied to a new case with a different model version. You want to keep the defaults for other stream settings for the new model version, and only change the few settings requested. Since, this is commonly done in simulations it's important to have a way to do this for the streams files. Doing this will make it easier to work with the streams than with the previous interface.

This relates to this issue in CDEPS...

ESCOMP/CDEPS#76

@ekluzek ekluzek added ty: enhancement tp: user interface Responsibility: CESM Responsibility to manage and accomplish this issue is through CESM labels Apr 30, 2021
@github-actions
Copy link
Contributor

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the Stale label May 11, 2023
@github-actions
Copy link
Contributor

This issue was closed because it has been stalled for 5 days with no activity.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale May 16, 2023
@mvertens
Copy link
Contributor

mvertens commented May 16, 2023

@JEdwards @ekluzek - I think we should keep this issue open. I think as CDEPS is used more widely it will be really nice to have this and at least keep it on our radar in the near term.

@ekluzek
Copy link
Contributor Author

ekluzek commented May 16, 2023

@mvertens since we now have the capability to change CDEPS streams using the user_nl_*_streams file -- is this one in cime still important? This issue was created before that capability, and I think maybe that capability takes care of what this issue was about. So my inclination is to leave this closed since the CDEPS ESCOMP/CDEPS#76 took care of it. What do you think about that?

@mvertens
Copy link
Contributor

@ekluzek - all good points. I was thinking more along the lines of being able to remove or add a stream - which we cannot currently do easily from the current capabilities. You can only change what is already there in the streams. However, looking more closely - you already raised this in ESCOMP/CDEPS#211.
So I am fine with keeping this issue closed.

@ekluzek ekluzek removed the Stale label May 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Responsibility: CESM Responsibility to manage and accomplish this issue is through CESM tp: user interface ty: enhancement
Projects
None yet
Development

No branches or pull requests

2 participants