Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@slam2049
Copy link

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!
  • If you've added a new tool - have you followed the pipeline conventions in the contribution docs- [ ] If necessary, also make a PR on the nf-core/nanoseq branch on the nf-core/test-datasets repository.
  • Make sure your code lints (nf-core lint).
  • Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
  • Usage Documentation in docs/usage.md is updated.
  • Output Documentation in docs/output.md is updated.
  • CHANGELOG.md is updated.
  • README.md is updated (including new tool citations and authors/contributors).

christopher-hakkaart and others added 30 commits February 24, 2023 22:12
With the updated biocontainer for m6anet being built, can update
the singularity and docker container with the latest version.

This version changed how m6anet is called.  It has `m6anet` and
the analysis as a parameter (`inference`, `dataprep`) instead of
`m6anet-run_inference` and `m6anet-dataprep` commands.

There are now two files for output from `m6anet inference`, but
the existing output of '*' captures those.  In the future, might
be nice to specify these individually and emit them as separate
outputs for downstream analysis.
I updated the samplesheet to output the filename that was attempted
to be used when looking for the fastq file.  When running the
pipeline with multiple samples, I was unable to determine which
sample was causing the samplesheet error until I added this code.
…ewSE_samplesheet

Samplesheet error message updated with more useful information if fastq can't be found.
@slam2049 slam2049 closed this Mar 26, 2025
@github-actions
Copy link

This PR is against the master branch ❌

  • Do not close this PR
  • Click Edit and change the base to dev
  • This CI test will remain failed until you push a new commit

Hi @slam2049,

It looks like this pull-request is has been made against the slam2049/nanoseq master branch.
The master branch on nf-core repositories should always contain code from the latest release.
Because of this, PRs to master are only allowed if they come from the slam2049/nanoseq dev branch.

You do not need to close this PR, you can change the target branch to dev by clicking the "Edit" button at the top of this page.
Note that even after this, the test will continue to show as failing until you push a new commit.

Thanks again for your contribution!

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.