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

Feature/jstassi/#75 #76 updates #85

Closed
wants to merge 4 commits into from

Conversation

gmao-jstassi
Copy link
Contributor

Updates to

  1. use logical directory path for FVROOT rather than physical (absolute) path
  2. add -partition and -qos options to monthly.yyyymm.pl.tmpl

@gmao-jstassi gmao-jstassi requested a review from a team as a code owner April 16, 2021 13:51
@gmao-jstassi gmao-jstassi added the 0 diff The changes in this pull request have verified to be zero-diff with the target branch. label Apr 16, 2021
Copy link
Collaborator

@rtodling rtodling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These look good by me.

@rtodling
Copy link
Collaborator

rtodling commented Jun 2, 2021

Joe, I am having problems w/ fvsetup in this - fvsetup itself seems to work - but I cannot xxdiff it (or diff it) w/ any other version of fvsetup since the one here comes up as a binary file! Really odd. Can you please look into this?

Copy link
Collaborator

@rtodling rtodling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes are already in develop - rt-GEOSadas-5_28_0_p1

@rtodling
Copy link
Collaborator

rtodling commented Jun 3, 2021

Changes are already in develop - rt-GEOSadas-5_28_0_p1

@rtodling rtodling closed this Jun 3, 2021
@mathomp4 mathomp4 deleted the feature/jstassi/#75-#76-updates branch June 3, 2021 20:05
@gmao-jstassi
Copy link
Contributor Author

gmao-jstassi commented Jun 8, 2021 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 diff The changes in this pull request have verified to be zero-diff with the target branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants