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

Deprecated Inheritance From std::iterator Didn't Make It To Release #176

Open
KurtAA42 opened this issue Jun 9, 2022 · 7 comments
Open

Comments

@KurtAA42
Copy link

KurtAA42 commented Jun 9, 2022

An UBLAS patch called: "Remove deprecated inheritance from std::iterator #97" didn't make it into the released version of Boost.

"std::iterator was deprecated in C++17 and removed in C++20. I replaced the inheritance with the 5 equivalent typedefs, even though they're not all used by ublas, for compatibility in case clients depend on them."

If I copy the file latest development version of the boost/numeric/ublas/detail/iterator.hpp into my boost directory it fixes the issue when building with VS 2019 C++20.

The fix was over 2 years ago and should have made it into Boost 1_75.

-Kurt

@j2bbayle
Copy link

Any updates on this?

Feels like this hack (adding back the deprecated and then removed std::iterator class) should be finally removed and the proposed fix (“Remove deprecated inheritance from std::iterator https://github.com/boostorg/ublas/pull/97”) included in the next upcoming release?

For users relying on C++17, lots of very irritating deprecation warnings appear when using ublas.

@j2bbayle
Copy link

j2bbayle commented Dec 1, 2023

Anyone?

@Kojoley
Copy link

Kojoley commented Apr 29, 2024

@pdimov @glenfe ublas has not had a single merge from develop to master in 5 years.

@glenfe
Copy link
Member

glenfe commented Apr 29, 2024

@Kojoley yes, I have complained about this before.

@bassoy Is uBLAS effectively unmaintained now? Do we need to find a new maintainer? If not, what is preventing a merge to master?

@pdimov
Copy link
Member

pdimov commented Apr 29, 2024

What I remember from previous conversations on the matter is that the uBLAS team have their own ideas of how the develop and master branches need to work - develop requires C++20, whereas master still doesn't.

But if so, someone needs to take care of cherry-picking fixes from develop to master.

@bassoy
Copy link
Collaborator

bassoy commented Apr 29, 2024

@pdimov and @glenfe . We have developed a lot on the develop branch a lot of which is not merged to the master. We have made very good progress on the tensor development. Sorry

I will take now full responsibility and start to solve the issues.

@bassoy
Copy link
Collaborator

bassoy commented May 5, 2024

@pdimov and @glenfe . We have developed a lot on the develop branch a lot of which is not merged to the master. We have made very good progress on the tensor development. Sorry

I will take now full responsibility and start to solve the issues.

Pull request opened #194.

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

6 participants