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 list of files needed to be read in to mizuroute.input_data_list #266

Closed
ekluzek opened this issue Mar 17, 2022 · 1 comment · Fixed by #391 · May be fixed by #267
Closed

Add list of files needed to be read in to mizuroute.input_data_list #266

ekluzek opened this issue Mar 17, 2022 · 1 comment · Fixed by #391 · May be fixed by #267
Labels
cesm-coupling For cesm coupling enhancement Improving usability, performance or other types of enhancements

Comments

@ekluzek
Copy link
Collaborator

ekluzek commented Mar 17, 2022

In the buildnml we need to add the list of files that mizuRoute is going to read to the file: for.input_data_list. Currently we've been relying on those files already being in place for mizuRoute to work. Adding this will allow cime to download them before the run is done. I noticed this starting to work on izumi, since it wasn't automatically downling the needed files.

@ekluzek ekluzek added enhancement Improving usability, performance or other types of enhancements cesm-coupling For cesm coupling labels Mar 17, 2022
@ekluzek
Copy link
Collaborator Author

ekluzek commented Mar 17, 2022

One of the reasons for this is that the mizuRoute control file is outside of the scope of the cime namelist mechanism generation. So it doesn't have a list of the needed files in the namelist_definition xml file. So the way to handle this now is to send the filename to the rof.input_data_list file directly. So that will need to be explicitly added in the buildnml for each filename that needs to be in the list.

@ekluzek ekluzek changed the title Add list of files needed to be read in to rof.input_data_list Add list of files needed to be read in to mizuroute.input_data_list May 31, 2023
@ekluzek ekluzek linked a pull request Jun 1, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cesm-coupling For cesm coupling enhancement Improving usability, performance or other types of enhancements
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants