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.

Sign up for GitHub

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

Jump to bottom

[pybind] Keep alive, refererence (tr.) usages and semnatics are confusing / possibly incorrect #18656

Open
EricCousineau-TRI opened this issue Jan 26, 2023 · 1 comment
Open

[pybind] Keep alive, refererence (tr.) usages and semnatics are confusing / possibly incorrect #18656

EricCousineau-TRI opened this issue Jan 26, 2023 · 1 comment
Assignees
@rpoyner-tri
Labels
component: pydrake Python API and its supporting Starlark macros priority: low type: bug

Comments

@EricCousineau-TRI
Copy link
Contributor

EricCousineau-TRI commented Jan 26, 2023 •
edited
Loading

See discussion in #18650

Given that we haven't seen acute errors, beyond things like #14387 that can be worked around, I won't plan to tackle it anytime soon. However, good to note!

\cc @SeanCurtis-TRI @jwnimmer-tri
@EricCousineau-TRI EricCousineau-TRI self-assigned this Jan 26, 2023
@jwnimmer-tri
Copy link
Collaborator

Mostly my disposition here overall is that we should switch to shared_ptr for our Python/C++ interfacing so that everything becomes a lot more obvious and natural, ala #13058 and related.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rpoyner-tri rpoyner-tri

Labels
component: pydrake Python API and its supporting Starlark macros priority: low type: bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@rpoyner-tri @jwnimmer-tri @EricCousineau-TRI