Skip to content

Resume Multiple Sequence Alignments#376

Merged
gtonkinhill merged 17 commits into
gtonkinhill:devfrom
nzmacalasdair:resume-aln
May 29, 2026
Merged

Resume Multiple Sequence Alignments#376
gtonkinhill merged 17 commits into
gtonkinhill:devfrom
nzmacalasdair:resume-aln

Conversation

@nzmacalasdair

Copy link
Copy Markdown
Collaborator

This adds the ability to resume MSAs which hit resource/runtime limits with panaroo-msa --resume, for both non-codon and codon alignments.

Does this by writing a small JSON with the alignment parameters, which is checked for when an alignment starts. Alignment module then looks for completed output in aligned_gene_sequences (or also aligned_protein_sequences for codon alignments) and skips things which are already there.

Done testing on two small datasets so far, seems to run without issues and produces 100% identical output. May be worth adding an experimental flag in the initial release however.

@nzmacalasdair nzmacalasdair requested a review from gtonkinhill May 12, 2026 12:51
@nzmacalasdair nzmacalasdair added the enhancement New feature or request label May 12, 2026
@nzmacalasdair nzmacalasdair changed the title Resume aln Resume Multiple Sequence Alignments May 12, 2026
@gtonkinhill

Copy link
Copy Markdown
Owner

Hi Neil,
This is currently hitting an error as the generate_pan_genome_alignment has been changed but not all the references to it have been updated.
Is there a reason it needs to have a 'core' parameter as all genes should be aligned when the 'pan' is specified by the user

@nzmacalasdair

Copy link
Copy Markdown
Collaborator Author

This is because the checkpoint file (which needs the threshold to be written so the resume can use the same threshold) is currently created within either generate_core_genome_alignment or generate_pan_genome_alignment.

There's no real need for it to be written within those functions though, it's inelegant and I will change that.

Are you happy with the helper function spam in generate_alignments.py? I could clean that up as well.

@gtonkinhill

Copy link
Copy Markdown
Owner

Thanks for that. Yes, happy with the spam 😄

@gtonkinhill gtonkinhill merged commit ea198ad into gtonkinhill:dev May 29, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants