Skip to content

Support more locales in R-CMD-check-occasional - #7869

Open
MichaelChirico wants to merge 12 commits into
tests-azerifrom
occasional-more-locales
Open

Support more locales in R-CMD-check-occasional#7869
MichaelChirico wants to merge 12 commits into
tests-azerifrom
occasional-more-locales

Conversation

@MichaelChirico

Copy link
Copy Markdown
Member

As surfaced during review, lv_LV is not quite enough to catch all possible collation issues.

Since this CI is only run "rarely", combinatorial explosion of settings is not so big a concern.

@github-actions

Copy link
Copy Markdown
  • HEAD=occasional-more-locales much slower for transform improved in #5493
  • HEAD=occasional-more-locales slower P<0.001 for DT[by] max regression fixed in #7480
    Comparison Plot

Generated via commit ef77a1e

Download link for the artifact containing the test results: ↓ atime-results.zip

Task Duration
R setup and installing dependencies 6 minutes and 26 seconds
Installing different package versions 12 minutes and 38 seconds
Running and plotting the test cases 5 minutes and 33 seconds

@MichaelChirico
MichaelChirico force-pushed the occasional-more-locales branch from a8575a9 to 9e6ca2e Compare July 31, 2026 02:26
@codecov

codecov Bot commented Jul 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.01%. Comparing base (e5916a1) to head (52b650c).

Additional details and impacted files
@@             Coverage Diff              @@
##           tests-azeri    #7869   +/-   ##
============================================
  Coverage        99.01%   99.01%           
============================================
  Files               88       88           
  Lines            17292    17292           
============================================
  Hits             17122    17122           
  Misses             170      170           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@MichaelChirico
MichaelChirico changed the base branch from fix-latvian-tests to tests-azeri July 31, 2026 03:41
@MichaelChirico

Copy link
Copy Markdown
Member Author

At least partly this is blocked by an upstream error that {R.oo} can't install in az_AZ:

HenrikBengtsson/R.oo#31

@MichaelChirico
MichaelChirico force-pushed the occasional-more-locales branch from 9e6ca2e to d47cd3c Compare July 31, 2026 07:04
@MichaelChirico MichaelChirico changed the title Occasional more locales Support more locales in R-CMD-check-occasional Jul 31, 2026
@MichaelChirico
MichaelChirico force-pushed the occasional-more-locales branch 2 times, most recently from 852132f to 07ca251 Compare July 31, 2026 15:04
@aitap

aitap commented Aug 2, 2026

Copy link
Copy Markdown
Member

Interesting. I've tried test.data.table() in all locales again (setting LC_ALL but also LANGUAGE=en; otherwise the failures are inconvenient to parse and many of those are probably due to R being translated while data.table is not) and the only remaining failures are tests 168.[1-3] in locales bo_CN, bo_IN, dz_BT, km_KH, shn_MM, tok, vi_VN: this is due to R bug 19117. Are the tests supposed to be still failing in CI?

@MichaelChirico
MichaelChirico force-pushed the occasional-more-locales branch from ecab787 to 625d570 Compare August 3, 2026 20:47
@MichaelChirico
MichaelChirico force-pushed the occasional-more-locales branch from 625d570 to df1d097 Compare August 3, 2026 20:59
@MichaelChirico
MichaelChirico force-pushed the occasional-more-locales branch from df1d097 to 3a54e8a Compare August 3, 2026 21:12
@MichaelChirico
MichaelChirico force-pushed the occasional-more-locales branch from 3a54e8a to 52b650c Compare August 3, 2026 22:46
@MichaelChirico

Copy link
Copy Markdown
Member Author

Are the tests supposed to be still failing in CI?

Green CI is always good. A behavioral test against R itself would be good of the form

if (strptime(...)) {
  cat("Tests 168.* skipped as affected by R bug #19117\n")
} else {
  ...
}

@MichaelChirico

Copy link
Copy Markdown
Member Author

Surprising that the same date works in all the problem locales!

passes = function() format(as.Date(strptime(strftime(as.Date('2000-11-01'), '%Y %d %b'), '%Y %d %b'))) == '2000-11-01'

for (ll in c("bo_CN", "bo_IN", "dz_BT", "km_KH", "shn_MM", "tok", "vi_VN"))
  print(withr::with_locale(list(LC_TIME=ll), passes()))
# all FALSE

# .utf8 suffix needed for https://bugs.r-project.org/show_bug.cgi?id=19117#c1
for (ll in c("en_US", "zh_CN.utf8", "lv_LV", "es_ES", "ru_RU.utf8", "pl_PL", "th_TH.utf8"))
  print(withr::with_locale(list(LC_TIME=ll), passes()))
# all TRUE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants