Repository navigation
Remove Intel records from python wrapper - #14875
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the Python wrapper documentation to remove “Intel” branding references, aligning the wrapper README with the “RealSense SDK 2.0” naming.
Changes:
- Rename the README title to remove “Intel”.
- Update the introductory sentence to remove “Intel”.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| Python Wrapper for RealSense SDK 2.0 | ||
| ========================================== | ||
|
|
||
| The python wrapper for Intel RealSense SDK 2.0 provides the C++ to Python binding required to access the SDK. | ||
| The python wrapper for RealSense SDK 2.0 provides the C++ to Python binding required to access the SDK. |
There was a problem hiding this comment.
The PR title suggests removing Intel branding from the Python wrapper, but there are still multiple "Intel RealSense" references under wrappers/python/examples (e.g., box_dimensioner_multicam/realsense_device_manager.py). Either update those remaining references or narrow/rename the PR to reflect that it only changes the README header.
AviaAv
left a comment
AviaAv
left a comment
There was a problem hiding this comment.
Looks good, but let's handle Copilot's comments as they seem relevant
|
|
||
| PYBIND11_MODULE(NAME, m) { | ||
| m.doc() = R"pbdoc( | ||
| LibrealsenseTM Python Bindings |
There was a problem hiding this comment.
TM is not needed in this file?
No description provided.