Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Casanovo Nextflow Workflow Documentation #391

Merged
merged 5 commits into from
Oct 30, 2024
Merged

Casanovo Nextflow Workflow Documentation #391

merged 5 commits into from
Oct 30, 2024

Conversation

Lilferrit
Copy link
Contributor

Added a Read the Docs page for the Casanovo Nextflow workflow.

@Lilferrit Lilferrit requested a review from bittremieux October 10, 2024 21:18
Copy link

codecov bot commented Oct 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.92%. Comparing base (396f838) to head (503f5c9).
Report is 9 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #391      +/-   ##
==========================================
- Coverage   94.37%   93.92%   -0.46%     
==========================================
  Files          13       13              
  Lines        1102     1102              
==========================================
- Hits         1040     1035       -5     
- Misses         62       67       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

docs/index.md Outdated Show resolved Hide resolved
docs/nextflow.md Show resolved Hide resolved
@Lilferrit Lilferrit requested a review from bittremieux October 15, 2024 23:23
Copy link
Collaborator

@bittremieux bittremieux left a comment

Choose a reason for hiding this comment

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

Looks good, just two minor changes.

docs/nextflow.md Outdated Show resolved Hide resolved
docs/nextflow.md Outdated
# Casanovo Nextflow Workflow

To simplify the process of setting up and running Casanovo, a dedicated [Nextflow](https://www.nextflow.io/) workflow is available.
In addition to automatic the installation of Casanovo and its dependencies, the Casanovo Nextflow workflow provides an automated mass spectrometry data pipeline that converts input data files to a Casanovo compatible format using [msconvert](https://bioinformaticshome.com/tools/proteomics/descriptions/msconvert.html#gsc.tab=0), infers peptide sequences using Casanovo, and (optionally) uploads the results to [Limelight](https://limelight-ms.org/) - a platform for sharing and visualizing proteomics results.
Copy link
Collaborator

Choose a reason for hiding this comment

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

todo: Official link for msConvert is: https://proteowizard.sourceforge.io/tools/msconvert.html

Copy link
Collaborator

@bittremieux bittremieux Oct 23, 2024

Choose a reason for hiding this comment

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

This still needs to be fixed. You used the link for msconvert for Nextflow instead, so now two links need to be updated.

@bittremieux bittremieux linked an issue Oct 24, 2024 that may be closed by this pull request
@bittremieux bittremieux merged commit e2290fa into dev Oct 30, 2024
7 checks passed
@bittremieux bittremieux deleted the nextflow-doc branch October 30, 2024 08:19
bittremieux added a commit that referenced this pull request Nov 21, 2024
* nextflow documentation

* nextflow nav prefix, more nextflow docs details

* grammatical fixes

* grammatical fixes

* fixed links
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.

Document nextflow workflow
2 participants