Enable backwards-compatibility with Eigen < 5#3338
Conversation
WardBrian
left a comment
There was a problem hiding this comment.
Seems fine if this is all that's needed to be more compatible.
Note that I think RcppEigen is behind partially because of us
|
That @WardBrian I was also about to comment that
is circular: I cannot update |
Jenkins Console Log Machine informationNo LSB modules are available. Distributor ID: Ubuntu Description: Ubuntu 20.04.3 LTS Release: 20.04 Codename: focalCPU: G++: Clang: |
Ahh I see, I'll have a look! |
Summary
The development version of
rstanis currently failing to build sinceRcppEigenis still a few versions behind (see this issue). The culprit is some of the new fancy ternary handling, which can be simply gated behind a preprocessor macro untilRcppEigenupdates.Tests
N/A - Existing tests should still pass
Side Effects
N/A
Release notes
Restored compatibility with Eigen < 5
Checklist
Copyright holder: Andrew Johnson
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)
the basic tests are passing
./runTests.py test/unit)make test-headers)make test-math-dependencies)make doxygen)make cpplint)the code is written in idiomatic C++ and changes are documented in the doxygen
the new changes are tested