-
Notifications
You must be signed in to change notification settings - Fork 991
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
add fread support for text= single line without \r or \n #4703
add fread support for text= single line without \r or \n #4703
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4703 +/- ##
==========================================
- Coverage 99.51% 99.51% -0.01%
==========================================
Files 76 76
Lines 14696 14694 -2
==========================================
- Hits 14625 14623 -2
Misses 71 71
Continue to review full report at Codecov.
|
Is this really fixed? I'm seeing this with 1.14.2:
|
@torbjorn this is fixed in the current dev-version. The added milestone might be misleading. You can update to the dev-version via This will also not be fixed with |
@ben-schwen @torbjorn once 1.14.4 is accepted on CRAN, I'll tag that in the patch-14.4 branch and then probably move forward with backporting bug fixes like this to 1.14.6 on that branch together with new features in dev that are stable and which shouldn't have any impact revdeps; e.g. the new |
@mattdowle It's true once I click on the milestone or check the milestone below it says 1.14.5. In the timeline of commits, mentions and merges it is still displayed as 1.14.1 (see following image) |
Thanks. Yes it's the same milestone that's since been renamed. Both the closed issue and the closed PR are marked 1.14.5 on the right though. Maybe in future then we should have a 'dev' or 'unstable' milestone and then move items to the release branch. Like R-devel does it. |
It's 1.14.8 now and this seems not to be fixed:
with a newline:
version:
|
@torbjorn this is fixed in the current dev version. |
Closes #4689.
fread(text = "a b c")
works now and produces andata.table
with 0 rows and 3 cols. Beforehand this resulted in an error because single line text without\r
or\n
was treated likefread(input = text)
.