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

Update COSIMA metadata.yaml files #104

Merged
merged 9 commits into from
Sep 28, 2023
Merged

Conversation

dougiesquire
Copy link
Collaborator

@dougiesquire dougiesquire commented Jul 18, 2023

I've asked that owners of the COSIMA runs in the catalog replace existing metadata.yaml files with those in /g/data/tm70/intake/metadata. In this PR, I'll update the paths as the replacements are made.

Closes #96

@codecov
Copy link

codecov bot commented Jul 18, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.31%. Comparing base (f7ca779) to head (6ba7e87).
Report is 102 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #104   +/-   ##
=======================================
  Coverage   97.31%   97.31%           
=======================================
  Files           9        9           
  Lines         522      522           
=======================================
  Hits          508      508           
  Misses         14       14           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@dougiesquire
Copy link
Collaborator Author

RTD build is now failing. This works locally with an old environment, but fails with the latest versions of everything. The error is occurring within the autoapi extension, but the version hasn't changed.

Extension error (autoapi.extension):
Handler <function run_autoapi at 0x103b55b20> for event 'builder-inited' threw an exception (exception: 'Module' object has no attribute 'doc')

@dougiesquire
Copy link
Collaborator Author

RTD build is now failing. This PR does not update the docs, and I've opened an issue here #118, so merging

@dougiesquire dougiesquire merged commit 5c564a8 into main Sep 28, 2023
14 checks passed
@dougiesquire dougiesquire deleted the 96_cosima_metadata_yaml branch September 28, 2023 05:49
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.

COSIMA metadata.yaml should be moved to experiment base directories
1 participant