-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Add Missing Python Bindings and Add/Update Docstrings #6429
Conversation
Thanks for submitting this pull request! The maintainers of this repository would appreciate if you could update the CHANGELOG.md based on your changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Rene. A couple more improvements to the documentation and we can merge.
cpp/pybind/visualization/gui/gui.cpp
Outdated
@@ -1550,6 +1550,11 @@ void pybind_gui_classes(py::module &m) { | |||
.def("remove_item", &TreeView::RemoveItem, | |||
"Removes an item and all its children (if any)") | |||
.def("clear", &TreeView::Clear, "Removes all items") | |||
.def("get_item", &TreeView::GetItem, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Document arguments here as well.
67bcb1b
to
c29a473
Compare
Type
Motivation and Context
Adds missing Python bindings to
Scene
class for controlling object properties such as culling (which provides partial solution to #6427) and toTreeView
class for retrieving selected widget (addresses #6413)Checklist:
python util/check_style.py --apply
to apply Open3D code styleto my code.
updated accordingly.
results (e.g. screenshots or numbers) here.
Description
This change is