Skip to content

Assemble modelCIF - DUMMY implementation without touching workflow wiring - #617

Draft
keiran-rowell-unsw wants to merge 186 commits into
nf-core:devfrom
Australian-Structural-Biology-Computing:assemble_modelcif
Draft

keiran-rowell-unsw wants to merge 186 commits into
nf-core:devfrom
Australian-Structural-Biology-Computing:assemble_modelcif

Conversation

@keiran-rowell-unsw

@keiran-rowell-unsw keiran-rowell-unsw commented May 20, 2026 •

Copy link
Copy Markdown
Contributor

Part implements an ASSEMBLE_MODELCIF{} process as sketched out in this project board.

  • DUMMY files used to populate ModelCIF fields
    • This PR avoids touching upstream workflow wiring intentionally: to merge first as test-suite then wire in values from pipeline carefully
    • DUMMY_METRIC.tsv now uses 'realistic' 20 residues values, and is synced with docs/output.md
  • Comprehensive nf-test suite:
    • Uses the RCSB CifCheck program to validate against the ModelArchive dictionary
    • Reject completely invalid .mmcif; distinguishes from .pdb; and uses a real ma-.cif deposition to validate fields
    • assemble_modelcif_X has ext.args config files to test creation of .bcif, or PAE embedded or linked
  • msa_tool used to specify .protocol.CoevolutionMSAStep() since the steps in-container aren't always inspectable
  • DUMMY_SOFTWARE_DETAILS used to handle minimal protocol ingest into .mmcif classes for now
  • populate_modelcif.py has a variety of _helper() local functions leading to a build_modelcif() function populated by argparse

DATABASES: databases will not be handled in .data.Datagroup.ReferenceDatabasein this PR. populate_modelcif.py is getting quite long already. Plus, it's a separate concept that can tie into the work done for reference dataset at NCI.

#575 might make this database handling easier, if considered valuable

DRAFT: still in draft as I'm LLM'ing and doc'ing through features and will got back for deeper inspection of .mmcif spec when ready to review

PR checklist

  • This comment contains a description of changes (with reason).
  • If you've fixed a bug or added code that should be tested, add tests!
  • Make sure your code lints (nf-core pipelines lint).
  • Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
  • CHANGELOG.md is updated.

keiran-rowell-unsw and others added 30 commits October 1, 2026 12:25
Sort ranked models, make the embedded config JSON injection-safe, de-duplicate model images and fix rendering for the ipTM/ipSAE/chainwise tables.
…Biology-Computing/proteinfold into assemble_modelcif
Co-authored-by: keiran-rowell-unsw <54380465+keiran-rowell-unsw@users.noreply.github.com>
Co-authored-by: keiran-rowell-unsw <54380465+keiran-rowell-unsw@users.noreply.github.com>
…Biology-Computing/proteinfold into assemble_modelcif
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.

5 participants