Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix package publish issues #87

Merged
merged 3 commits into from
Feb 17, 2025
Merged

Fix package publish issues #87

merged 3 commits into from
Feb 17, 2025

Conversation

sokovninn
Copy link
Member

@sokovninn sokovninn commented Feb 17, 2025 •
edited
Loading

Purpose

Update dependencies and deployment configuration for better compatibility and CI/CD reliability.

Specification

  • Added unofficial sam2 pypi package to requirements.txt.
  • Changed base image in Dockerfile.
  • Fixed tag in GHCR deploy action.

Dependencies & Potential Impact

  • Potential compatibility issues with the new base image.
  • Updated dependencies may affect package resolution.

@sokovninn sokovninn requested review from HonzaCuhel and a team February 17, 2025 14:09
@sokovninn sokovninn self-assigned this Feb 17, 2025
@sokovninn sokovninn requested review from kozlov721, klemen1999, tersekmatija and conorsim and removed request for a team February 17, 2025 14:09
@sokovninn sokovninn merged commit 183506d into main Feb 17, 2025
2 checks passed
@sokovninn sokovninn deleted the chore/package-publish-issues branch February 17, 2025 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kozlov721 kozlov721 kozlov721 approved these changes

@klemen1999 klemen1999 klemen1999 approved these changes

@HonzaCuhel HonzaCuhel Awaiting requested review from HonzaCuhel

@tersekmatija tersekmatija Awaiting requested review from tersekmatija tersekmatija was automatically assigned from luxonis/ml-reviewers

@conorsim conorsim Awaiting requested review from conorsim conorsim was automatically assigned from luxonis/ml-reviewers

Assignees

@sokovninn sokovninn

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sokovninn @kozlov721 @klemen1999