-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat: update core schema v2 for alignments and mesh #273
Conversation
# Conflicts: # ingestion_tools/dataset_configs/template.yaml # schema/ingestion_config/v1.0.0/codegen/ingestion_config_models.py
- add enums for alignment types and format
…smith/997-ctf-alignment # Conflicts: # ingestion_tools/dataset_configs/template.yaml # schema/core/v1.1.0/metadata.yaml
# Conflicts: # ingestion_tools/dataset_configs/10003.yaml # ingestion_tools/dataset_configs/10006.yaml # ingestion_tools/dataset_configs/10436.yaml # ingestion_tools/dataset_configs/10439.yaml
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please run make-build-ingestion-config
to update the pydantic models before merging!
@@ -594,6 +612,9 @@ classes: | |||
is_visualization_default: | |||
exact_mappings: | |||
- cdp-common:annotation_source_file_is_visualization_default | |||
is_portal_standard: | |||
exact_mappings: | |||
- cdp-common:annotation_source_file_is_portal_standard |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We had a discussion about the is_portal_standard
field the other day and iirc It's only supposed to be present on Tomogram
and Alignment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cc @uermel
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove from v1 and v2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just marking this for changes while we get clarification on the crossreferences/dates fields on tomograms, and the is_portal_standard
field on annotations. The rest lgtm!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved once the standardized field is removed from annotations
# Conflicts: # ingestion_tools/dataset_configs/10002.yaml # ingestion_tools/dataset_configs/10004.yaml # ingestion_tools/dataset_configs/10008.yaml # ingestion_tools/dataset_configs/10009.yaml # ingestion_tools/dataset_configs/10426.yaml # ingestion_tools/dataset_configs/10436.yaml # ingestion_tools/dataset_configs/10437.yaml # ingestion_tools/dataset_configs/10438.yaml # ingestion_tools/dataset_configs/10439.yaml # ingestion_tools/dataset_configs/deposition_10301.yaml # ingestion_tools/dataset_configs/deposition_10303.yaml # ingestion_tools/dataset_configs/deposition_10304.yaml # ingestion_tools/dataset_configs/deposition_10308.yaml # ingestion_tools/scripts/schema_migration/transform_ingestion_configs.py
# Conflicts: # ingestion_tools/dataset_configs/10000.yaml # ingestion_tools/dataset_configs/10001.yaml # ingestion_tools/dataset_configs/10002.yaml # ingestion_tools/dataset_configs/10003.yaml # ingestion_tools/dataset_configs/10004.yaml # ingestion_tools/dataset_configs/10005.yaml # ingestion_tools/dataset_configs/10006.yaml # ingestion_tools/dataset_configs/10007.yaml # ingestion_tools/dataset_configs/10008.yaml # ingestion_tools/dataset_configs/10009.yaml # ingestion_tools/dataset_configs/10010.yaml # ingestion_tools/dataset_configs/10301.yaml # ingestion_tools/dataset_configs/10436.yaml # ingestion_tools/dataset_configs/10437.yaml # ingestion_tools/dataset_configs/10438.yaml # ingestion_tools/dataset_configs/10439.yaml # ingestion_tools/dataset_configs/deposition_10301.yaml # ingestion_tools/dataset_configs/deposition_10302.yaml # ingestion_tools/dataset_configs/deposition_10303.yaml # ingestion_tools/dataset_configs/deposition_10304.yaml # ingestion_tools/dataset_configs/deposition_10305.yaml # ingestion_tools/dataset_configs/deposition_10308.yaml # ingestion_tools/scripts/schema_migration/migrate_v1_1_0.py # schema/core/v1.1.0/codegen/metadata_materialized.yaml # schema/core/v1.1.0/codegen/metadata_models.py # schema/core/v1.1.0/metadata.yaml # schema/ingestion_config/v1.0.0/codegen/ingestion_config_models.py # schema/ingestion_config/v1.0.0/codegen/ingestion_config_models.schema.json
Description
Update core schema v2 to match the changes in for v1 from chanzuckerberg/cryoet-data-portal#997