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
Test 1590 robust to locale #2813
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2813 +/- ##
=======================================
Coverage 93.45% 93.45%
=======================================
Files 61 61
Lines 12367 12367
=======================================
Hits 11558 11558
Misses 809 809 Continue to review full report at Codecov.
|
@jangorecki I'll merge without review given CRAN release is pressing. Please check post-merge and we can always revert if needed. |
oy = if (length(oy <- forderv(y))) oy else seq_along(y) | ||
test(1590.5, oy, order(y)) | ||
# data.table::forderv is encoding-aware and independent of locale | ||
# [ Aside: data.table needs to be independent of locale because keys/indexes depend on a sort order. If a data.table is stored on disk with a key |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
note to self to add this to FAQ #2664
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
did fix the problem
Closes #2771