Update MAGs-generation to 0.5#1221
Conversation
Test Results (powered by Planemo)Test Summary
Errored Tests
|
Test Results (powered by Planemo)Test Summary
Errored Tests
|
|
strange the stated tool is installed in |
|
ping @paulzierep @bebatut |
Test Results (powered by Planemo)Test Summary
Errored Tests
|
Test Results (powered by Planemo)Test Summary
Errored Tests
|
|
@SantaMcCloud can you replace the nest tool with the apply rules tool ? it can also nest by one level. |
| "color": "yellow", | ||
| "data": { | ||
| "title": "Binner (MaxBin2)" | ||
| "title": "COMEBin" |
There was a problem hiding this comment.
can you add the pick parameter tools after comebin, this is always needed if a tool is optional
There was a problem hiding this comment.
yes i can do this i forgot to add it sorry
| Read length (CONCOCT): '100' | ||
| Environment for the built-in model (SemiBin): global | ||
| Contamination weight (Binette): '2' | ||
| Contamination weight (Binette): '100' |
There was a problem hiding this comment.
If i use a low weight Binette favor complete bins which i dont want to happen because of dRep need more bins as input so i try to create a lot of "useless" bin for testing
| CheckM2 Database: 1.0.2 | ||
| Minimum MAG completeness percentage: '1' | ||
| Maximum MAG contamination percentage: '25' | ||
| Maximum MAG contamination percentage: '1' |
There was a problem hiding this comment.
this will probably fail, I would go for 99 for the test
There was a problem hiding this comment.
The test i did run this didnt failed since i wanted to create more MAGs so dRep get more input therefor i set the limit low.
Co-authored-by: paulzierep <paul.zierep@googlemail.com>
I can replace it yes. Maybe the tool is install now i will wait to check if this is needed or not |
Test Results (powered by Planemo)Test Summary
Errored Tests
|
1 similar comment
Test Results (powered by Planemo)Test Summary
Errored Tests
|
I did change it and i hope i did it right. I also add the pick value tool. The marked changed from the box title the one you did marked is correct so i did not change this. |
Test Results (powered by Planemo)Test Summary
Errored Tests
|
Test Results (powered by Planemo)Test Summary
Errored Tests
|
1 similar comment
Test Results (powered by Planemo)Test Summary
Errored Tests
|
Test Results (powered by Planemo)Test Summary
Errored Tests
|
Test Results (powered by Planemo)Test Summary
Errored Tests
|
|
This is the current error cut out of the logs: Not sure why it does fail since testing it manually there is no such error and the WF runs without any problem on ping @paulzierep for updateing you! |
FOR CONTRIBUTOR:
FOR REVIEWERS:
This workflow does/runs/performs … xyz … to generate/analyze/etc …namefield should be human readable (spaces are fine, no underscore, dash only where spelling dictates it), no abbreviation unless generally understood-) over underscore (_), prefer all lowercase. Folder becomes repository in iwc-workflows organization and is included in TRS id