search

nf-core / deepvariant

Please consider using/contributing to https://github.com/nf-core/sarek
https://nf-co.re
MIT License
40 stars 21 forks source link

Bam.size() fix #7

Closed apeltzer closed 5 years ago

apeltzer commented 5 years ago

This should fix the issue with bam.size() according to paolo...

https://github.com/nextflow-io/nextflow/issues/922#issuecomment-437895942

PhilPalmer commented 5 years ago

Thanks @apeltzer, unfortunately, while the job can now be submitted the build fails on the make_examples process with the same error we had before

Caused by:
  nextflow.util.MemoryUnit cannot be cast to java.base/java.lang.Long

Exception in thread "Task submitter" java.lang.IllegalArgumentException: Cannot compare java.lang.Long with value '3,925,783' and nextflow.util.MemoryUnit with value '1 GB'

I'll try and see if there's another fix for it

apeltzer commented 5 years ago

There is since ~30mins ago: https://github.com/nextflow-io/nextflow/issues/921 Should make it easier, though we need to bump minimum nxf version to 18.11.0-edge