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

TA3 Schema Integration and Metadata Parsing #36

Merged
merged 21 commits into from
Jun 21, 2024
Merged

TA3 Schema Integration and Metadata Parsing #36

merged 21 commits into from
Jun 21, 2024

Conversation

ajmuelle
Copy link
Contributor

This isn't fully implemented (in particular the ingestion of NetCDF data) but we wanted to get the discussion going regarding how to merge in our new schemas for TA3 data along with the ingestion script.

@ajmuelle
Copy link
Contributor Author

ajmuelle commented Jun 18, 2024

populate_ta3_schema.py (to be renamed) now lives at the cdr repo.

@@ -0,0 +1,18 @@
from cdr_schemas.ta3_input import StackMetaData
Copy link
Contributor

Choose a reason for hiding this comment

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

Should be changed now?

@@ -0,0 +1,18 @@
from cdr_schemas.ta3_input import StackMetaData
from cdr_schemas.ta3_model import CMAModel
Copy link
Contributor

Choose a reason for hiding this comment

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

Should be changed now?

@scotthaleen
Copy link
Contributor

We have a docs generator script https://github.com/DARPA-CRITICALMAAS/cdr_schemas/blob/main/dev/docs.py#L59

You can delete the ERD generator and update the docs script.

https://github.com/DARPA-CRITICALMAAS/cdr_schemas?tab=readme-ov-file#development

If you run

poetry install --with docs
poetry run docs

It should generate the ERD in README. It can be a bit finicky

@scotthaleen scotthaleen merged commit 3734554 into main Jun 21, 2024
1 check passed
@scotthaleen scotthaleen deleted the ta3 branch June 21, 2024 13:53
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.

3 participants