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

JOSS Review: Review Documentation #8

Closed
rcannood opened this issue Jan 11, 2025 · 0 comments
Closed

JOSS Review: Review Documentation #8

rcannood opened this issue Jan 11, 2025 · 0 comments

Comments

@rcannood
Copy link
Contributor

rcannood commented Jan 11, 2025

I'm reviewing the JOSS submission of AliNe in openjournals/joss-reviews#7545.

Example usage

Do the authors include examples of how to use the software (ideally to solve real-world analysis problems).

The Usage section in the readme currently contains:

  • Check available options with nextflow run ... --help
  • Command will look like that: nextflow run ... -profile docker <rest of parameters>.

Could you add an example of a command with actual input parameters?

Example:

nextflow run Juke34/AliNe \
  -r v1.0.0 \
  -profile docker \
  --reads https://github.com/Juke34/AliNe/raw/refs/heads/main/test/illumina/yeast_R1.fastq.gz \
  --genome https://raw.githubusercontent.com/Juke34/AliNe/refs/heads/main/test/yeast.fa \
  --read_type short_single \
  --aligner bbmap,bowtie2,bwaaln,bwamem,bwasw,graphmap2,hisat2,minimap2,ngmlr,nucmer,star,subread,sublong,tophat2 \
  --trimming_fastp \
  --star_options "--genomeSAindexNbases 9" \
  --multiqc_config https://raw.githubusercontent.com/Juke34/AliNe/refs/heads/main/config/multiqc_conf.yml

(The above is only possible after #7 is fixed)

or:

git clone https://github.com/Juke34/AliNe --depth=1

nextflow run Juke34/AliNe \
  -r v1.0.0 \
  -profile docker \
  --reads AliNe/test/illumina \
  --genome AliNe/test/yeast.fa \
  --read_type short_single \
  --aligner bbmap,bowtie2,bwaaln,bwamem,bwasw,graphmap2,hisat2,minimap2,ngmlr,nucmer,star,subread,sublong,tophat2 \
  --trimming_fastp \
  --star_options "--genomeSAindexNbases 9" \
  --multiqc_config AliNe/config/multiqc_conf.yml

This would greatly help users run the workflow :)

@Juke34 Juke34 closed this as completed in ae576ce Jan 13, 2025
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

No branches or pull requests

1 participant