Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compatibility with older versions of boost #76

Open
MichaelChirico opened this issue Mar 17, 2022 · 2 comments
Open

Compatibility with older versions of boost #76

MichaelChirico opened this issue Mar 17, 2022 · 2 comments

Comments

@MichaelChirico
Copy link
Contributor

Hi, we are stuck with an old (1.67) version of boost. I was able to get colourvalues working (compiles and passes tests) by making pretty minor edits here:

#include <boost/math/interpolators/detail/cardinal_cubic_b_spline_detail.hpp>

It was straightforward to swap out cardinal_cubic_b_spline_detail.hpp for cubic_b_spline_detail.hpp. Is there any known impact on behavior from doing so? It should also be relatively easy to have an #ifdef block to use the right header depending on the version.

@dcooley
Copy link
Collaborator

dcooley commented Mar 17, 2022

I don't think I can confirm one way or another what the impact will be if I'm honest, as I don't know the differences between the implementations. I can't imagine it will be noticable from an end-user perspective though.

I'm happy if you want to make a PR with the #ifdef so it works for you.

@MichaelChirico
Copy link
Contributor Author

I did see this in the original commit for the cardinal_ version:

boostorg/math@caf1883

(but I also can't make heads or tails of it). Such a PR would be great! I've edited our local copy to work, but would be helpful for others (?) that might be stuck with old boost to get colourvalues working.

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

No branches or pull requests

2 participants