-
Notifications
You must be signed in to change notification settings - Fork 463
New tool Addition: Chopper a tool for fast processing (filtering, trimming, ..) of long reads #7046
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @lsterck for the addition, some review comments inline.
right, I think I know what the problem is ... on it. |
OK, I've ran the local planemo test a couple of times and after the edit I pushed before it seems to consistently pass (which was indeed not the case before) but not here apparently. What am I missing? Any ideas how to resolve this? |
I could remove the gzip test but I would prefer to keep it if in any way possible (just to also check the gzip option) |
Maybe you can have try using asserts as per here: tools-iuc/tools/pbtk/bam2fastx.xml Line 46 in eca92f6
|
FOR CONTRIBUTOR:
Hi,
this is a pull request to add the Chopper tool to the iuc toolshed.
I'm happy to amend any review comments.
best regards,
lieven