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.
| @@ -1,7 +1,7 @@ | |||
| Python Wrapper for Intel RealSense SDK 2.0 | |||
| Python Wrapper for RealSense SDK 2.0 | |||
| ========================================== | |||
There was a problem hiding this comment.
The reStructuredText section underline length no longer matches the updated title. Sphinx/docutils can emit warnings or render oddly when the underline is longer/shorter than the title; please adjust the '=' count to match the title length.
| ========================================== | |
| ==================================== |
| 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.
| ========================================== | ||
|
|
||
| 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.
In this sentence, "python" should be capitalized as "Python" (also consistent with the document title).
| The python wrapper for 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. |
No description provided.