Skip to content

Add support for loading PTM/PCM data - #37

Open
brindakv wants to merge 3 commits into
masterfrom
ptm_pcm
Open

Add support for loading PTM/PCM data#37
brindakv wants to merge 3 commits into
masterfrom
ptm_pcm

Conversation

@brindakv

@brindakv brindakv commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

This PR

  • Updates PDBx/mmCIF dictionary to version 5.424
  • Adds support for pdbx_modification_feature and pdbx_chem_comp_pcm
  • Includes search metadata for PTM/PCM
  • Removes pdbx_vrpt_summary_entity_fit_to_map from pdbx_core_nonpolymer_entity_instance schema
  • Updates modified_monomer polymer entity feature to non_standard_monomer feature

@brindakv
brindakv requested a review from valasatava September 4, 2026 15:30

@valasatava valasatava left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you, @brindakv. Just few comments

}
]
},
"uniprot_generic_ptm_accession": {},

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This attribute and the one below miss schema definitions

"context": "dictionary"
},
{
"text": "Modified Residue Part",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Let's not add this to UI in this first iteration. We had some doubts when we discussed. Thanks!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks @valasatava. Do you still want it indexed?

@brindakv
brindakv requested a review from valasatava September 5, 2026 03:37
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