Skip to content

merge sort arg document better#4427

Merged
mattdowle merged 13 commits intomasterfrom
merge-order
Aug 2, 2021
Merged

merge sort arg document better#4427
mattdowle merged 13 commits intomasterfrom
merge-order

Conversation

@jangorecki
Copy link
Copy Markdown
Member

explain difference of sorting NAs in data.table merge

closes #2574
closes #2594

@codecov
Copy link
Copy Markdown

codecov bot commented May 4, 2020

Codecov Report

Merging #4427 (be246bf) into master (2791043) will increase coverage by 0.13%.
The diff coverage is n/a.

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

@@            Coverage Diff             @@
##           master    #4427      +/-   ##
==========================================
+ Coverage   99.47%   99.60%   +0.13%     
==========================================
  Files          75       72       -3     
  Lines       14808    13917     -891     
==========================================
- Hits        14730    13862     -868     
+ Misses         78       55      -23     
Impacted Files Coverage Δ
src/fmelt.c 99.00% <0.00%> (-1.00%) ⬇️
src/ijoin.c 95.29% <0.00%> (-0.18%) ⬇️
src/fsort.c 95.83% <0.00%> (-0.10%) ⬇️
R/like.R 100.00% <0.00%> (ø)
src/cj.c 100.00% <0.00%> (ø)
R/fcast.R 100.00% <0.00%> (ø)
R/fmelt.R 100.00% <0.00%> (ø)
R/frank.R 100.00% <0.00%> (ø)
R/fread.R 100.00% <0.00%> (ø)
R/merge.R 100.00% <0.00%> (ø)
... and 47 more

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 f5fa5ec...c5e68be. Read the comment docs.

Comment thread man/merge.Rd
@mattdowle mattdowle added this to the 1.14.1 milestone Aug 2, 2021
@mattdowle mattdowle merged commit dff60a8 into master Aug 2, 2021
@mattdowle mattdowle deleted the merge-order branch August 2, 2021 23:17
mattdowle added a commit that referenced this pull request Oct 9, 2022
…eems it was fixed in R 3.5.0. And the nabble link doesn't seem to be working either, so removing the paragraph solves that too.
@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.

forderv orders NA's different than base::order by default sort = FALSE does't generate equivalent results with data.frame.merge

3 participants