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

(HOLD) RO-2578: Schema updates for Data API 2.0 #124

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

piehld
Copy link
Collaborator

@piehld piehld commented Jul 17, 2023

RO-2595: Remove internal _struct.pdbx_descriptor from schema

RO-170:

RO-3099:

  • Remove pdbx_reference_molecule_details from bird_chem_comp_core
  • Note that this field is already no longer populated in ExDB

@piehld piehld requested a review from brindakv July 17, 2023 15:06
@piehld
Copy link
Collaborator Author

piehld commented Jul 17, 2023

Maybe I should bundle this with the changes needed for RO-3099 as well...

@piehld
Copy link
Collaborator Author

piehld commented Jul 17, 2023

And this...RO-170

Copy link
Contributor

@brindakv brindakv left a comment

Choose a reason for hiding this comment

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

Looks good @piehld.

Feel free to add changes based on RO-3099 and/or RO-170.

piehld added 2 commits July 17, 2023 17:17
- Remove ordinal attribute from rcsb_polymer_instance_feature, rcsb_nonpolymer_instance_feature, rcsb_branched_instance_feature
- Remove reference_scheme from all polymer_entity, polymer_entity_instance, branched_entity, and branched_entity_instance feature objects

RO-3099:
- Remove pdbx_reference_molecule_details from bird_chem_comp_core
@piehld piehld changed the title RO-2595: Remove internal _struct.pdbx_descriptor from schema RO-2578: Schema updates for Data API 2.0 Jul 17, 2023
@piehld
Copy link
Collaborator Author

piehld commented Jul 17, 2023

Thanks @brindakv, I've incorporated the changes required by the other tickets and updated the schema files. Are you able to have one more look?

@piehld
Copy link
Collaborator Author

piehld commented Jul 18, 2023

NOTE: Do not merge until ready for Data API 2.0 release (see comments on Jira here).

When the time comes, will need to create a parallel PR to rcsb-json-schema as well.

@piehld piehld changed the title RO-2578: Schema updates for Data API 2.0 (HOLD) RO-2578: Schema updates for Data API 2.0 Jul 18, 2023
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.

2 participants