Skip to content

Return full table for !all(FALSE) with subset#3829

Merged
mattdowle merged 3 commits intomasterfrom
with_all_not_false
Sep 11, 2019
Merged

Return full table for !all(FALSE) with subset#3829
mattdowle merged 3 commits intomasterfrom
with_all_not_false

Conversation

@MichaelChirico
Copy link
Copy Markdown
Member

@MichaelChirico MichaelChirico commented Sep 6, 2019

Closes #3013
Closes #2917

Problem as correctly identified by @DavidArenburg in the issue

@codecov
Copy link
Copy Markdown

codecov bot commented Sep 6, 2019

Codecov Report

Merging #3829 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3829   +/-   ##
=======================================
  Coverage   99.42%   99.42%           
=======================================
  Files          71       71           
  Lines       13416    13416           
=======================================
  Hits        13339    13339           
  Misses         77       77
Impacted Files Coverage Δ
R/data.table.R 100% <100%> (ø) ⬆️

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 ef59b35...d03bdca. Read the comment docs.

@codecov
Copy link
Copy Markdown

codecov bot commented Sep 6, 2019

Codecov Report

Merging #3829 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3829   +/-   ##
=======================================
  Coverage   99.42%   99.42%           
=======================================
  Files          71       71           
  Lines       13399    13399           
=======================================
  Hits        13322    13322           
  Misses         77       77
Impacted Files Coverage Δ
R/data.table.R 100% <100%> (ø) ⬆️

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 27f7516...ca196b0. Read the comment docs.

Copy link
Copy Markdown
Member

@jangorecki jangorecki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mattdowle mattdowle added this to the 1.12.4 milestone Sep 11, 2019
@mattdowle mattdowle merged commit a2d4f27 into master Sep 11, 2019
@mattdowle mattdowle deleted the with_all_not_false branch September 11, 2019 21:28
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.

Strange column selection with logical condition ! combined with a logical call in j seem to have precedence bug

3 participants