Conversation
There was a problem hiding this comment.
Pull request overview
This PR merges version 1.0.1.33 into master, primarily updating the D4xx kernel module version and improving test infrastructure alongside documentation updates for JetPack 6.0 and 6.2.
Changes:
- Updated kernel module version from 1.0.1.32 to 1.0.1.33
- Added new FPS test infrastructure and pytest configuration
- Improved error handling and byte string decoding in test files
- Added JetPack 6.2 deployment and installation scripts
- Split JetPack 6.x documentation into separate 6.0 and 6.2 guides
Reviewed changes
Copilot reviewed 17 out of 18 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| kernel/realsense/d4xx.c | Updated module version to 1.0.1.33 |
| test/test_fw_version.py | Improved error handling and added proper string decoding |
| test/test_fps.py | New comprehensive FPS testing functionality |
| test/run_ci.py | Enhanced exception reporting and added missing sys.exit call |
| test/pytest.ini | Added pytest configuration with d457 marker |
| test/install.tegra.artifacts.sh | New script for installing Tegra artifacts |
| test/groovy/LRS_libci_pipeline.groovy | New Jenkins pipeline configuration |
| setup_workspace.sh | Added local tarball caching support and fixed variable references |
| scripts/install_to_kernel_6.2.sh | New installation script for kernel 6.2 |
| scripts/deploy_kernel_6.2.sh | New deployment script for kernel 6.2 |
| scripts/aggregate_kernel_6.x.sh | New aggregation script for kernel modules |
| nvidia-oot/6.0/0003-Fix-y12i-calibration-stream.patch | Updated Y12I frame error workaround with better implementation |
| build_all.sh | Fixed source directory variable reference |
| apply_patches.sh | Improved handling of sources directory naming |
| README_JP6.2.md | Updated deployment instructions and fixed documentation links |
| README_JP6.0.md | New comprehensive setup guide for JetPack 6.0 |
| README.md | Updated links to point to version-specific documentation |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ----<CUT>---- | ||
| LABEL JetsonIO | ||
| MENU LABEL Custom Header Config: <CSI Jetson RealSense Camera D457> | ||
| LINUX /boot/dev/Image |
There was a problem hiding this comment.
The LINUX path shows /boot/dev/Image which suggests a custom boot folder setup, but this isn't explained in the preceding instructions. The deployment instructions in step 1 copy Image to /boot/ directly, not /boot/dev/. This inconsistency between the example configuration and the installation instructions could confuse users.
| LINUX /boot/dev/Image | |
| LINUX /boot/Image |
|
|
||
| ``` | ||
| # Configuration files | ||
| tar czf rootfs.tar.gz -C images/6.2/rootfs boot lib |
There was a problem hiding this comment.
The path references images/6.2/rootfs but this is the JetPack 6.0 documentation file. This should be images/6.0/rootfs to match the version being documented.
| tar czf rootfs.tar.gz -C images/6.2/rootfs boot lib | |
| tar czf rootfs.tar.gz -C images/6.0/rootfs boot lib |
No description provided.