GeezerVison

Hi ngme2 team,

Eigen 5.0.0, which we are trying to bring to CRAN in an updated RcppEigen, requires a small update to CHOLMOD from your package. (Also note that they changed the versioning scheme so Eigen 5.0.0 corresponds to 3.5.0 under the old scheme.) See RcppEigen issue 151 for all the changes related to this upgrade, and see the Eigen 5 release notes for their changes.

We now require a LinkingTo: Matrix (you already imported Matrix so not a new dependency) and that leads to a possible dual includion of CHOLMOD headers, so a small local shim CholmodSupport (see below) is added to your package which ensures include order is fine.

With these changes it builds under both the CRAN version of RcppEigen (using Eigen 3.4.0.2) and the development version of RcppEigen (from our repo, and r-universe) with Eigen 5.0.1.

So if you could apply this patch and get the update to CRAN before we update, a new RcppEigen would not break your package. If you don't update it we would get breakage 😞 but this PR overcomes this. We have been at this update for a long time as there were other issues in affected packages so the upload is not really imminent -- but getting closer. If you can get this to CRAN over the next few weeks we would appreciate it. 'Some time' in the near future will work, you can see at RcppEigen issue 151 how the other changes come in.

If you have any questions please do not hesitate to reach out.

Thanks!