From f583f84f488b1ffdc1af6c4ed94cd00e007742da Mon Sep 17 00:00:00 2001 From: merliseclyde Date: Tue, 17 Sep 2024 17:09:37 -0400 Subject: [PATCH] update cran-comments and NEWS for submission to CRAN --- NEWS.md | 4 ++++ cran-comments.md | 18 +++++++++++++++--- 2 files changed, 19 insertions(+), 3 deletions(-) diff --git a/NEWS.md b/NEWS.md index 72cb2d87..d181dad3 100644 --- a/NEWS.md +++ b/NEWS.md @@ -1,3 +1,7 @@ +# BAS 1.7.3 + +* reverted back to code using `SETLENGTH` (issue #82) to address stack imbalance issues seen in interactive checks and not flagged in `R CMD check`. + # BAS 1.7.2 * added `method="AMCMC"` for `bas.lm` to use adaptive independent Metropolis Hastings for sampling diff --git a/cran-comments.md b/cran-comments.md index e5227b0d..96bf2545 100644 --- a/cran-comments.md +++ b/cran-comments.md @@ -2,11 +2,16 @@ ## Submission reason -Addressed Strict and Notes for Rdevel on the [Checks page at CRAN] (https://cran.r-project.org/web/checks/check_results_BAS.html): +BAS 1.7.2 which was submitted on 9/16 to +address the "Strict" and "Notes" issues for R-Devel on the [Checks page at CRAN] (https://cran.r-project.org/web/checks/check_results_BAS.html): - Removed legacy definitions of ‘PI’ and ‘Free’ and replaced with‘M_PI’ and ‘R_Free’ to comply with ‘STRICT_R_HEADERS’ (issue #81) to prevent package removal after 9/23/2024 -- Removed non-API calls to SETLENGTH (issue #82) +- Removed non-API calls to `SETLENGTH` (issue #82) + +The latter change inadvertently is leading to stack imbalances that are not caught in R CMD checks on R-Devel or R-Release with M1Mac but appear in interactive sessions or under checks with valgrind. + +To eliminate the stack imbalance the src code has reverted back to using `SETLENGTH` to truncate over-allocated vectors prior to assigning to the returned SEXP and UNPROTECTING the memory before function exit. (As the number of entries is random, it is not possible to pre-determine the vector size, but just an upper limit.) This code has been in the package since 2018 and further work is still needed to update without creating other memory issues in large problems in practice. ## Test environments @@ -18,8 +23,15 @@ Addressed Strict and Notes for Rdevel on the [Checks page at CRAN] (https://cran ## R CMD check results for this submission * Mac, Windows, Ubunto - 0 error | 0 warnings | 0 notes + 0 error | 0 warnings | 1 notes + +* checking compiled code ... NOTE +File 'BAS/libs/x64/BAS.dll': + Found non-API call to R: 'SETLENGTH' + +Compiled code should not call non-API entry points in R. +This is under development and I ask that CRAN allow this update despite the NOTE to allow the package to be retained on CRAN. The non-API calls identified by `Strict` have been fixed. ## Reverse Dependencies