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

Unknown method invocation toBytes on String type #142

Closed
heuermh opened this issue Nov 28, 2020 · 5 comments
Closed

Unknown method invocation toBytes on String type #142

heuermh opened this issue Nov 28, 2020 · 5 comments
Labels
bug Something isn't working

Comments

@heuermh
Copy link
Contributor

heuermh commented Nov 28, 2020

$ nextflow run main.nf -profile test,docker
...
[b2/a44cc7] process > output_documentation                                           [100%] 1 of 1 ✔
Unknown method invocation `toBytes` on String type -- Did you mean?
  getBytes

 -- Check script 'main.nf' at line: 3457 or see '.nextflow.log' file for more details
@drpatelh
Copy link
Member

Thanks for posting this @heuermh Some more info would be useful though 😅 I am assuming you are running the dev version of the pipeline because that points to

def smail_fields = [ email: email_address, subject: subject, email_txt: email_txt, email_html: email_html, projectDir: "$projectDir", mqcFile: mqc_report, mqcMaxSize: params.max_multiqc_email_size.toBytes() ]
as being the issue here. However, this is a little odd because the GitHub Actions CI tests are run in exactly the same way and passed on the last PR...

@heuermh
Copy link
Contributor Author

heuermh commented Nov 28, 2020

Yep, am still in investigation mode, dev from git HEAD fails for me on OSX (#143) but not on linux (which succeeds but notes this issue).

@drpatelh
Copy link
Member

drpatelh commented Nov 28, 2020

Ok. This is quite scary because the CI tests passed but do indeed show the same error you just reported https://github.com/nf-core/viralrecon/runs/1453984332#step:7:215 . Its because I changed the type of params.max_multiqc_email_size in nextflow.config from a Nextflow memory unit to a string to deal with standardising things for the new JSON schema 🤦

@drpatelh
Copy link
Member

@drpatelh drpatelh added the bug Something isn't working label Nov 30, 2020
heuermh pushed a commit that referenced this issue Nov 30, 2020
heuermh added a commit that referenced this issue Nov 30, 2020
@heuermh
Copy link
Contributor Author

heuermh commented Nov 30, 2020

Fixed by #144

@heuermh heuermh closed this as completed Nov 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants