-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
problems with new Eigen API #41523
Comments
@RalphAS: thanks for your constructive critics. I agree, that this PR was kind of not complete, when I posted it.
I will have time to work out a PR at the earliest in a few weeks, hopefully, when I will be back from hospital. You are predestined for reviewing that. |
For keywords perhaps "left", "right", "econdition", "vcondition" would be good. (I summon @dpsanders for advice.) Best wishes for successful procedures and gentle & speedy recovery. |
We have very little time before the 1.7 release and we since the fixes most likely will involve API changes, I think the best way to proceed here is to revert the original PR and get a new version of this merged in the 1.8 cycle. Hence, I've opened #41578. |
There are several.
The text was updated successfully, but these errors were encountered: