[ITEP-83716] [Dependabot] Updates /autocalibration#733
[ITEP-83716] [Dependabot] Updates /autocalibration#733
Conversation
…ibration (#722) Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dmytro Yermolenko <dmytro.yermolenko@intel.com>
Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dmytro Yermolenko <dmytro.yermolenko@intel.com>
dmytroye
left a comment
There was a problem hiding this comment.
Don't merge before autocalibration tests fix.
There was a problem hiding this comment.
Pull request overview
This PR updates dependency versions in the autocalibration module's runtime requirements, specifically upgrading intel_extension_for_pytorch from 2.7.0 to 2.8.0 and trimesh from 4.7.4 to 4.10.1 to keep dependencies current.
- Updated intel_extension_for_pytorch to version 2.8.0
- Updated trimesh to version 4.10.1
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Patryk Iracki <patryk.iracki@intel.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dmytro Yermolenko <dmytro.yermolenko@intel.com>
Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Patryk Iracki <patryk.iracki@intel.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dmytro Yermolenko <dmytro.yermolenko@intel.com>
Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Patryk Iracki <patryk.iracki@intel.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dmytro Yermolenko <dmytro.yermolenko@intel.com>
… in /autocalibration (#853)
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…n-edge-platform/scenescape into dependabot-updates-autocalib
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
Comments suppressed due to low confidence (1)
autocalibration/Dockerfile:93
- The base image has been pinned to a specific SHA256 digest. This is not consistent with other services in the codebase (controller, manager, etc.) which use tag-based references like
python:3.11-slim-bookworm.
While SHA256 pinning provides stronger immutability guarantees, it has several drawbacks:
- It's harder to review since the digest is opaque
- It makes updates more complex and doesn't benefit from security patches to the base image
- It's inconsistent with the rest of the codebase
Consider reverting to the tag-based reference python:3.11-slim-bookworm to maintain consistency with other services, unless there's a specific security or reproducibility requirement for this service.
FROM python:3.11-slim-bookworm@sha256:65a93d69fa75478d554f4ad27c85c1e69fa184956261b4301ebaf6dbb0a3543d AS autocalibration-runtime
📝 Description
Provide a clear summary of the changes and the context behind them. Describe what was changed, why it was needed, and how the changes address the issue or add value.
✨ Type of Change
Select the type of change your PR introduces:
🧪 Testing Scenarios
Describe how the changes were tested and how reviewers can test them too:
✅ Checklist
Before submitting the PR, ensure the following: