Repository navigation
Conversation
…ual Zenodo deposits
Documentation build overview
23 files changed ·
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem and resulting behavior
MetaPathways 3.5.2 moves workflow orchestration and reference construction to Nextflow while retaining the existing commands and sample outputs. The complete
analysis_wfcommand runs multiple assemblies, optional reads and genome maps through annotation, abundance, community/MAG PGDB inference and combined reporting on a local server or Slurm.This PR targets
devand should remain draft while the local/HPC benchmarks and independent review are pending. Production promotion will be a separatedev→masterPR. No release tag or registry publication is part of this PR.Changes to review
build_ptcreates and registers a private Apptainer image from a licensed installer, applies official vendor patches and validates BLAST support. Optional matching MetaCyc reference preparation includes indexes and mapping tables. PGDB staging supplies sequences and coordinates to community and MAG builds; optional MAG failures retain explicit outcomes.Validation
a320bb4f6bf81f62a1a560d609b49ceaab3f7c7d; subsequent PR-preparation changes are documentation only. Record the final tested PR commit before sign-off.--skip_ptools; it is historical evidence, not a fresh licensed-PGDB or final-commit test.Remaining merge gates
release_tagandsource_run_id; retain package artifacts, optionally Docker/SIF validation and security reports, and add run links here.Expected no-pathway or optional MAG outcomes must remain distinguishable from task failures. The final installed Conda/container artifacts require their own checks; older candidate builds do not validate this exact revision. Benchmark results are pending and are not claimed as passed.
Follow the tester checklist and release process. Keep licensed installers, private images, MetaCyc data, credentials and full benchmark outputs out of the PR.