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 support for selecting different driver. #1988

Merged
merged 2 commits into from
Oct 30, 2017

Conversation

jgfouca
Copy link
Contributor

@jgfouca jgfouca commented Oct 24, 2017

  1. Add --driver flag to create_newcase
  2. Add support for V$driver test opt to create_test
  3. Remove places where mct was hardcoded
  4. Add support for driver selection to case.configure, case.create

Lots of XML changes will be needed in the models to truly support
different drivers. This PR just removes the core infrastructure as
blocker for this effort.

Test suite: scripts_regression_tests
Test baseline:
Test namelist changes:
Test status: bit for bit

Fixes #1781

User interface changes?: Yes, --driver added to create_newcase, new _V$driver testopt

Update gh-pages html (Y/N)?: N

Code review: @jedwards4b @mvertens

1) Add --driver flag to create_newcase
2) Add support for _V$driver_ test opt to create_test
3) Remove places where mct was hardcoded
4) Add support for driver selection to case.configure, case.create

Lots of XML changes will be needed in the models to truly support
different drivers. This PR just removes the core infrastructure as
blocker for this effort.
@rljacob
Copy link
Member

rljacob commented Oct 24, 2017

What are the XML changes needed? Does this supply all the functionality @mvertens implemented in her NUOPC branch?

@jgfouca
Copy link
Contributor Author

jgfouca commented Oct 24, 2017

I'm not familiar with her branch. @mvertens , can you chime in here?

@mvertens
Copy link
Contributor

@jgfouca - the forked noupc repository is https://github.com/apcraig/cime and the branch is nuopcB. This branch now also has the -Vnuopc as an option and its up to date with esmci/master hash 0e5cee1 - which is a few hashes after cime5.4.0-alpha.04.

@jedwards4b
Copy link
Contributor

@jgfouca can we merge this? It may have conflicts with @mvertens fork but can we reconcile them we she is ready to merge again?

@jgfouca
Copy link
Contributor Author

jgfouca commented Oct 30, 2017

@jedwards4b yeah, that's fine.

@jgfouca jgfouca merged commit f99c174 into master Oct 30, 2017
@jgfouca jgfouca deleted the jgfouca/add_driver_selection_support branch October 30, 2017 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants