Skip to content

Upkeep to satisfy CRAN checks and requests#15

Merged
RLumSK merged 7 commits intoR-Lum:masterfrom
mcol:upkeep
Sep 17, 2025
Merged

Upkeep to satisfy CRAN checks and requests#15
RLumSK merged 7 commits intoR-Lum:masterfrom
mcol:upkeep

Conversation

@mcol
Copy link
Member

@mcol mcol commented Sep 17, 2025

This does the following:

  • Update the OS requested in the github workflow
  • Fix notes from R CMD check
  • Update roxygen2 documentation
  • Stop requiring C++11 to satisfy CRAN request
  • Fix compilation warnings due to the new RCppArmadillo

This should avoid long queueing times for the old machines.
This fixes these notes:

N  checking R code for possible problems (4.8s)
   Found if() conditions comparing class() to string:
   File ‘RLumModel/R/read_SEQ2R.R’: if (class(file) != "character") ...
   File ‘RLumModel/R/simulate_CW_OSL.R’: if (class(n) != "RLum.Results") ...
   File ‘RLumModel/R/simulate_LM_OSL.R’: if (class(n) != "RLum.Results") ...
   File ‘RLumModel/R/simulate_RF.R’: if (class(n) != "RLum.Results") ...
   File ‘RLumModel/R/simulate_RF_and_heating.R’: if (class(n) != "RLum.Results") ...
   File ‘RLumModel/R/simulate_TL.R’: if (class(n) != "RLum.Results") ...
   File ‘RLumModel/R/simulate_heating.R’: if (class(n) != "RLum.Results") ...
   File ‘RLumModel/R/simulate_illumination.R’: if (class(n) != "RLum.Results") ...
   File ‘RLumModel/R/simulate_pause.R’: if (class(n) != "RLum.Results") ...
   Use inherits() (or maybe is()) instead.
This should fix this error when running the github workflow:

 Error: This request has been automatically failed because it uses a
 deprecated version of `actions/cache: v2`. Please update your workflow
 to use v3/v4 of actions/cache to avoid interruptions. Learn more:
 https://github.blog/changelog/2024-12-05-notice-of-upcoming-releases-and-breaking-changes-for-github-actions/#actions-cache-v1-v2-and-actions-toolkit-cache-package-closing-down
This is an attempt to fix the following:

Run rcmdcheck::rcmdcheck(args = c("--no-manual", "--as-cran"),
                         error_on = "warning", check_dir = "check")
Error in dyn.load(file, DLLpath = DLLpath, ...) :
  unable to load shared object '/home/runner/work/_temp/Library/curl/libs/curl.so':
  libssl.so.1.1: cannot open shared object file: No such file or directory
Calls: loadNamespace ... asNamespace -> loadNamespace -> library.dynam -> dyn.load
@RLumSK RLumSK merged commit fedfc85 into R-Lum:master Sep 17, 2025
2 of 5 checks passed
RLumSK added a commit that referenced this pull request Sep 17, 2025
This reverts commit fedfc85, reversing
changes made to 2e6b71b.
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.

2 participants