Skip to content

Merge 1.0.2.15 to master - #382

Closed
ymodlin wants to merge 38 commits into
masterfrom
r/1.0.2.15
Closed

Merge 1.0.2.15 to master#382
ymodlin wants to merge 38 commits into
masterfrom
r/1.0.2.15

Conversation

@ymodlin

@ymodlin ymodlin commented Feb 25, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

sareluzi and others added 30 commits January 29, 2026 13:11
* Fix readme file to remove Intel

* Update Security.md
…347)

Tracked-on RSDSO 21089: Device did not reset after executing the hardware reset script
- Update GitHub repo URLs from IntelRealSense to realsenseai
- Update store URLs from store.intelrealsense.com to store.realsenseai.com
- Remove "Intel®" branding from titles and descriptions
- Replace broken DES457 store links with text-only mentions
* Refactor nvethernetrm symlink handling for JetPack 6.x to ensure proper directory removal before linking
* Adding XU control for camera sync command

* Fix sync_mode control range not applied for device-specific
…er several iterations streaming start/stop with pipeline. (#356)

* Implement retry mechanism for streaming status checks and enhance SERDES pipe management

* Add I2C retry mechanism

* Bump module version to 1.0.2.6
- Removed obsolete test scripts: run_ci.py, test_fps.py, test_fw_version.py.
- Updated conftest.py to include new resolution discovery functions for depth, RGB, and IR.
- Enhanced constants.py to define known driver names for better driver validation.
- Improved discovery.py to support camera discovery via symlinks on Tegra platforms.
- Modified test_controls.py to add tests for auto-exposure mode switching and manual exposure control.
- Updated test_discovery.py to validate driver names against known constants.
- Enhanced test_error_handling.py to gracefully handle unsupported resolutions and zero FPS settings.
- Improved test_metadata.py to configure metadata capture with error handling for fixed formats.
- Added concurrent streaming tests in test_streaming.py for depth, RGB, and IR streams.
- Updated device.py to handle ioctl errors gracefully for tegra-video driver.
- Added new ioctl constants in ioctls.py for exposure control.
* Software i2c addr aliasing

To allow for deserializer boards which do not have the I2C mux
that aliases 8 i2c addr (30 - 37) to i2c 0, we define all devices
under i2c@0 with virtual addr 0x1b-0x1d, and in the driver override
them with 0x1a

* Remove dead IPU6 code

Signed-off-by: ejgoldik <ehud.joseph.goldik@intel.com>
Signed-off-by: ejgoldik <ehud.joseph.goldik@realsenseai.com>
* Disable metadata for Calib format
* Added overlays for calib mode
* bump version to 1.0.2.10

Signed-off-by: ejgoldik <ehud.joseph.goldik@realsenseai.com>
Co-authored-by: Nikolai-L <nlyskov@realsenseai.com>
Co-authored-by: ymodlin <ymodlin@realsenseai.com>
* Impement skills and agents for claude

* Add V4L2 test skill and runner script for remote execution on Jetson devices
- Add GitHub Actions workflow that runs V4L2 pytest suite on a
  self-hosted Jetson runner (driver reload, sanity check, pytest,
  dmesg collection, artifact upload)
- Add setup script for installing GitHub Actions runner on Jetson
  with proper labels and passwordless sudo for driver operations
- Supports manual trigger with test filter and auto-trigger on
  push/PR to kernel/realsense or test/v4l2_test paths

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Allow read-only remote commands via sshpass (cat, ls, find, grep, dtc,
dpkg, nvidia-smi, etc.) while denying destructive operations (rm -rf,
mkfs, reboot, shutdown, dd). Also includes standard local tool permissions
for git, build system, deployment, and common CLI utilities.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Oneshot only when calib

Signed-off-by: ejgoldik <ehud.joseph.goldik@realsenseai.com>
Comment thread apply_patches.sh
fi

# Stage all modified files after patching
git -C "sources_$SOURCES_VERSION/$D4XX_SRC_DST" add -A

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Repeated git staging/config/commit sequences for multiple subrepos were added; consolidate into a loop or helper that iterates target directories and applies git add/config/commit to avoid duplicated maintenance points.

Details

✨ AI Reasoning
​The change introduces multiple near-identical git configuration and staging/commit steps that perform the same operation (set user.name/user.email, add all, commit) against several sub-repositories. These blocks all accomplish the same business function and differ only in the directory path being targeted. This duplication increases maintenance burden because any change to how subrepos are prepared (for example changing commit message, author handling, or the add/commit behavior) would need to be made in several places. Consolidating into a loop or helper that iterates over the existing directories would reduce repetition and make future changes safer and clearer.

🔧 How do I fix it?
Delete extra code. Extract repeated code sequences into reusable functions or methods. Use loops or data structures to eliminate repetitive patterns.

Reply @AikidoSec feedback: [FEEDBACK] to get better review comments in the future.
Reply @AikidoSec ignore: [REASON] to ignore this issue.
More info

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AikidoSec ignore: not production code, internal script

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Based on your feedback, we ignored this isssue because of the following reason:

not production code, internal script

fi

echo "Copying kernel files for JetPack ${JETPACK_VERSION}..."
if [ "${JETPACK_VERSION}" = "5.0.2" ]; then

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Multiple repeated echo + sudo cp lines perform the same copy operation for different files. Consolidate into a loop or helper to avoid duplicated copy logic.

Details

✨ AI Reasoning
​A newly added script contains repeated, near-identical sequences that echo a command then run sudo cp for multiple kernel/artifact files. Repeating the same cp + echo pattern for several files means future changes (path/target adjustments, error handling) would need updates in many places, increasing maintenance risk. Consolidating these into loops or helper functions would reduce duplication and centralize behavior.

🔧 How do I fix it?
Delete extra code. Extract repeated code sequences into reusable functions or methods. Use loops or data structures to eliminate repetitive patterns.

Reply @AikidoSec feedback: [FEEDBACK] to get better review comments in the future.
Reply @AikidoSec ignore: [REASON] to ignore this issue.
More info

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AikidoSec ignore: not production code, internal script

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Based on your feedback, we ignored this isssue because of the following reason:

not production code, internal script

@ymodlin ymodlin closed this May 18, 2026
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.

6 participants