Skip to content

Comments

block g[[ on non-atomic input#4161

Merged
mattdowle merged 2 commits intomasterfrom
gforce-atomic
Jan 8, 2020
Merged

block g[[ on non-atomic input#4161
mattdowle merged 2 commits intomasterfrom
gforce-atomic

Conversation

@MichaelChirico
Copy link
Member

Closes #4159 (when combined with #4160)

Again, not sure this is the ideal approach, but it gets the job done. Better would be to write a proper g[[ for list input but that's not in the cards from me for now.

@MichaelChirico MichaelChirico mentioned this pull request Jan 5, 2020
17 tasks
@codecov
Copy link

codecov bot commented Jan 5, 2020

Codecov Report

Merging #4161 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4161      +/-   ##
==========================================
+ Coverage   99.54%   99.54%   +<.01%     
==========================================
  Files          72       72              
  Lines       13918    13924       +6     
==========================================
+ Hits        13854    13860       +6     
  Misses         64       64
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 a387663...a016a11. Read the comment docs.

@mattdowle mattdowle added this to the 1.12.9 milestone Jan 8, 2020
@mattdowle mattdowle merged commit 5833fd9 into master Jan 8, 2020
@mattdowle mattdowle deleted the gforce-atomic branch January 8, 2020 01:23
@jangorecki jangorecki modified the milestones: 1.12.11, 1.12.9 May 26, 2020
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.

returning .SD by group doesn't unlock .SD; and GForce [[ non-atomic type causes trouble

3 participants