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

Full support for Eigen::Map(ped) types #1402

Open
psiha opened this issue May 18, 2018 · 1 comment
Open

Full support for Eigen::Map(ped) types #1402

psiha opened this issue May 18, 2018 · 1 comment

Comments

@psiha
Copy link

psiha commented May 18, 2018

(and Eigen::TensorMap)

AFAICT pybind11 supports Eigen maps ('views') as return values and not as arguments (I get a compilation error about a delete function and in a nearby comment it says that maps are not supported as arguments but without an explanation as to why) and I don't see why this should be the case. In the documentation you recommend the use of the Eigen::Ref template. However Refs are slightly heavier and at the same time less versatile (in certain contexts I care about, like adapting/viewing memory of arbitrary yet 'compatible' objects as Eigen types, e.g. OpenCV Mats) than Maps and my codebase is already based on (that is to say littered with:) Maps so changing all that is not a (happy) option.
Could you add this support?

@bstaletic
Copy link
Collaborator

Hi. The problem is that, at this point, I don't think any of the active maintainers knows Eigen. We would need help, in the form of a pull request, preferably.

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