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 explicity bathymetry.nc to MOM_input #173

Merged
merged 1 commit into from
May 4, 2024

Conversation

ashjbarnes
Copy link
Collaborator

@ashjbarnes ashjbarnes commented May 4, 2024

The default filename for topography is topog.nc. Need to overwrite it to bathymetry.nc in our naming conventions.

closes #172

Copy link
Contributor

@navidcy navidcy left a comment

Choose a reason for hiding this comment

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

lgtm but did you actually try it and it runs?

@navidcy
Copy link
Contributor

navidcy commented May 4, 2024

Given that you stated you tested it, let’s merge and make a patch release v0.5.2

navidcy

This comment was marked as duplicate.

@navidcy navidcy merged commit a0c1fb4 into main May 4, 2024
6 checks passed
@navidcy navidcy deleted the explicit_bathymtry-filename branch May 4, 2024 18:13
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.

Need to provide TOPO_FILE arg in MOM_input otherwise MOM6 looks for bathymetry in topog.nc
2 participants