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

numpy_dtype_user: Pave way for downstream libs to define user dtypes #12

Conversation

EricCousineau-TRI
Copy link
Collaborator

@EricCousineau-TRI EricCousineau-TRI commented Mar 16, 2018

At present, this PR builds and runs in Bazel, but segfaults in CMake:
https://github.com/EricCousineau-TRI/repro/blob/0993f47d027e7e5f3212e55b5c658f9b600b3152/python/pybind11/BUILD#L225-L237
Will fix that tomorrow.

Relates RobotLocomotion/drake#8315 RobotLocomotion/drake#8116

  • Suppress errors in C++11 (for now, I don't want to spend too much time supporting it)
  • Fix compiler warnings
  • Fix Python3 tests
  • Submit upstream PR to Drake and ensure it doesn't fall flat

\cc @m-chaturvedi @soonho-tri @RussTedrake

Related upstream PRs:


This change is Reviewable

@EricCousineau-TRI EricCousineau-TRI force-pushed the feature/numpy_dtype_user branch 2 times, most recently from 524c2b7 to 11250dc Compare March 22, 2018 08:24
@EricCousineau-TRI EricCousineau-TRI force-pushed the feature/numpy_dtype_user branch 7 times, most recently from ae0beaa to e33c5ff Compare April 9, 2018 23:13
@EricCousineau-TRI EricCousineau-TRI force-pushed the feature/numpy_dtype_user branch from e33c5ff to 6d90c77 Compare April 9, 2018 23:28
@EricCousineau-TRI EricCousineau-TRI force-pushed the feature/numpy_dtype_user branch 3 times, most recently from 824a89b to e712b18 Compare April 14, 2018 06:28
@EricCousineau-TRI EricCousineau-TRI force-pushed the feature/numpy_dtype_user branch 3 times, most recently from 35fbeb8 to 3a4d863 Compare May 11, 2018 18:10
@EricCousineau-TRI EricCousineau-TRI force-pushed the feature/numpy_dtype_user branch 2 times, most recently from 2cccd6b to 906b5f2 Compare September 30, 2018 03:51
@EricCousineau-TRI EricCousineau-TRI changed the title numpy_dtype_user: Allow custom user dtypes to be defined numpy_dtype_user: Pave way for downstream libs to define user dtypes Sep 30, 2018
@EricCousineau-TRI EricCousineau-TRI force-pushed the feature/numpy_dtype_user branch from 906b5f2 to 7b4c254 Compare April 26, 2019 03:29
@EricCousineau-TRI
Copy link
Collaborator Author

Closing (3 yrs old)

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

Successfully merging this pull request may close these issues.

1 participant