Skip to content

adding the phrog command - #39

Merged
linsalrob merged 1 commit into
mainfrom
rdme
Aug 6, 2026
Merged

adding the phrog command#39
linsalrob merged 1 commit into
mainfrom
rdme

Conversation

@linsalrob

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI lite review requested due to automatic review settings August 6, 2026 00:44
@linsalrob
linsalrob merged commit f16df99 into main Aug 6, 2026
1 check passed
@linsalrob
linsalrob deleted the rdme branch August 6, 2026 00:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds documentation for running a new PHROGs annotation/counting step as part of the pawsey_shortread workflow, and adjusts README section formatting to include it in the numbered pipeline.

Changes:

  • Promotes step 2 into a proper Markdown heading.
  • Introduces a new “PHROGs” section with example Slurm submissions (and renumbers subsequent sections).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.



# 2. Check the names of your files.
## 2. Check the names of your files.
Comment on lines +218 to +219
We can use the mmseqs analysis to find PHROGs. We have a mapping between the UniProt IDs and the PHOG IDs which we use to map
similarities.
Comment on lines +224 to +225
PHROGJOB=$(sbatch --parsable --dependency=afterok:$MMSEQSJOB --array=1-$NUM_R1_READS:1 $PAWSEY_SRC/mmseqs_add_phrog_function.slurm)
sbatch --parsable --dependency=afterok:$PHROGJOB $PAWSEY_SRC/count_phrog_functions.slurm
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