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

add sortgff and gff3trimfasta modules #64

Open
wants to merge 3 commits into
base: jbrowse_indexer
Choose a base branch
from

Conversation

SantiagoSanchezF
Copy link

No description provided.

Copy link
Member

@mberacochea mberacochea left a comment

Choose a reason for hiding this comment

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

Keep them coming :)

I left a few comments to address

modules/ebi-metagenomics/gff3trimfasta/main.nf Outdated Show resolved Hide resolved
modules/ebi-metagenomics/gff3trimfasta/main.nf Outdated Show resolved Hide resolved
modules/ebi-metagenomics/gff3trimfasta/main.nf Outdated Show resolved Hide resolved
description: |
Groovy Map containing sample information
e.g. [ id:'test', single_end:false ]
- tab:
Copy link
Member

Choose a reason for hiding this comment

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

tab... gff seems more appropiate as a the name

Copy link
Author

Choose a reason for hiding this comment

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

gff is used for output file

Copy link
Member

Choose a reason for hiding this comment

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

sorterd_gff should be used for the gff then

modules/ebi-metagenomics/jbrowse/sortgff/main.nf Outdated Show resolved Hide resolved
modules/ebi-metagenomics/jbrowse/sortgff/main.nf Outdated Show resolved Hide resolved
description: |
Groovy Map containing sample information
e.g. [ id:'test', single_end:false ]
- tab:
Copy link
Member

Choose a reason for hiding this comment

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

I would use gff instead of tab

Copy link
Author

Choose a reason for hiding this comment

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

I'll leave is as tab, as I'm using gff for the output. Changed the descrition to be more specific

Copy link
Member

Choose a reason for hiding this comment

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

it should be gff, tab doesn't really convey what the input is... I would change the output to tbe sorted_gff or similar

modules/ebi-metagenomics/jbrowse/sortgff/meta.yml Outdated Show resolved Hide resolved
modules/ebi-metagenomics/jbrowse/sortgff/main.nf Outdated Show resolved Hide resolved
@mberacochea
Copy link
Member

Please, run the pre-commits too. There area few linting issues

container 'quay.io/biocontainers/gawk:4.1.3--0'

input:
tuple val(meta), path(tab)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
tuple val(meta), path(tab)
tuple val(meta), path(gff)

tuple val(meta), path(tab)

output:
tuple val(meta), path("*._trimmed.gff"), optional: true, emit: gff
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
tuple val(meta), path("*._trimmed.gff"), optional: true, emit: gff
tuple val(meta), path("*._trimmed.gff"), optional: true, emit: trimmed_gff

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