-
Notifications
You must be signed in to change notification settings - Fork 29
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
Bug in getSummaryFile due to bug in readr #2
Comments
Hi Xin, thank you so much for pointing this out to me! I will replace read_tsv by read_delim today and come back to you. Many thanks! |
Hi Xin, I just fixed the problem. Please find all news here: https://github.com/HajkD/biomartr/blob/master/NEWS.md#bug-fixes I now fixed all affected functions: I will submit the new version of biomartr to CRAN in one or two weeks. But you can already install via devtools from github. Again, many thanks for your help. I appreciate it a lot! |
Hi,
There is a bug in readr read_tsv and it only parses partial bacteria assembly_summary file. For example, there is over 80,000 rows and only 40,000 + rows loaded due to the quote. But read_tsv lacks of qoute option and it can be fixed by read_delim which you can set quote option.
The text was updated successfully, but these errors were encountered: