GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — For Eigen5 switch MappsedSparseMatrix to Map<SparseMatrix> by eddelbuettel · Pull Request #91 · covaruber/sommer

Conversation

@eddelbuettel

@eddelbuettel eddelbuettel commented Oct 30, 2026

Copy link
Copy Markdown
Contributor

Hello sommer team,

Eigen 5.0.0, which we are trying to bring to CRAN in an updated RcppEigen, did away with Eigen::MappedSparseMatrix. (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.

For MappedSparseMatrix, a change to Map works -- which was done in a good handful of places. No other changes are made in the PR. As with the other packages, this permits compilation under existing (Rcpp)Eigen and the candidate release permitting the upgrade. Because I generally run a newer Rcpp here and didn't want anything to spill I edited RcppExports.cpp rather than regenerate it. You may to want to overwrite this with a newly generated one, the effect ought to be identical.

So if you could apply this PR 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, and we are now looking at mid-to-late November. 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.

Thanks!

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@eddelbuettel
For Eigen5 switch MappsedSparseMatrix to Map<SparseMatrix>
5184644
@eddelbuettel eddelbuettel mentioned this pull request Oct 30, 2026
Transition to Eigen 5.0.* RcppCore/RcppEigen#151
Open
44 of 50 tasks
@covaruber
covaruber merged commit eb9fcf8 into covaruber:unified Oct 30, 2026
@covaruber

covaruber commented Oct 31, 2026

Copy link
Copy Markdown
Owner

I hope it is ok if I submit in 10 days since I just submitted to CRAN this week and they may not allow me to submit so soon. Thanks a lot for the heads up and for the pull request.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@eddelbuettel

eddelbuettel commented Oct 31, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor Author

Sure, and I know that nag -- it is actually two. One for 'at least seven days since last upload' (which one can override for bugfixes etc, but I usually try to honor this too) and 'no more than six in six months' which I have to watch for e.g. RcppArmadillo as Armadillo is quite active.

That said, a few more days is no issue. We are looking at mid-to-late November so that should cover it.

covaruber reacted with thumbs up emoji covaruber reacted with laugh emoji
All reactions
  • 👍 1 reaction
  • 😄 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@covaruber

covaruber commented Oct 31, 2026

Copy link
Copy Markdown
Owner

Thanks Dirk, and a second thanks for maintaining such an important set of cornerstone packages for the community :) have a great weekend.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@eddelbuettel
eddelbuettel deleted the feature/eigen5 branch October 31, 2026 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@eddelbuettel @covaruber
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.