Skip to content

fread support na.strings="0" and na.strings="1"#5196

Merged
mattdowle merged 1 commit intomasterfrom
fread_nastring=0
Oct 15, 2021
Merged

fread support na.strings="0" and na.strings="1"#5196
mattdowle merged 1 commit intomasterfrom
fread_nastring=0

Conversation

@ben-schwen
Copy link
Copy Markdown
Member

Closes #2927

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 4, 2021

Codecov Report

Merging #5196 (ab7e6f1) into master (8f0afcb) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head ab7e6f1 differs from pull request most recent head 2d0ba3e. Consider uploading reports for the commit 2d0ba3e to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master    #5196   +/-   ##
=======================================
  Coverage   99.38%   99.38%           
=======================================
  Files          77       77           
  Lines       14540    14541    +1     
=======================================
+ Hits        14451    14452    +1     
  Misses         89       89           
Impacted Files Coverage Δ
src/fread.c 99.40% <100.00%> (+<0.01%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8f0afcb...2d0ba3e. Read the comment docs.

@mattdowle mattdowle added this to the 1.14.3 milestone Oct 15, 2021
@mattdowle mattdowle merged commit 6265d30 into master Oct 15, 2021
@mattdowle mattdowle deleted the fread_nastring=0 branch October 15, 2021 09:00
@jangorecki jangorecki modified the milestones: 1.14.9, 1.15.0 Oct 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

na.strings="0" is not permitted in fread

3 participants