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

metrics.compute_missing_metrics kwarg 'periodic' not used #82

Closed
callumrollo opened this issue Feb 14, 2024 · 0 comments · Fixed by #92
Closed

metrics.compute_missing_metrics kwarg 'periodic' not used #82

callumrollo opened this issue Feb 14, 2024 · 0 comments · Fixed by #92

Comments

@callumrollo
Copy link
Contributor

Describe the bug

Function metrics.compute_missing_metrics has a kwarg periodic=False which is not used. Within this function a call is made to xgcm.Grid

grid = xgcm.Grid(ds, periodic=False)

Perhaps this should be

grid = xgcm.Grid(ds, periodic=periodic)

Using the kwarg from the metrics.compute_missing_metrics . Otherwise the kwarg periodic should be removed from metrics.compute_missing_metrics

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 a pull request may close this issue.

1 participant