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

Remove ImplicitKeypointBBoxHead #98

Merged
merged 3 commits into from
Oct 7, 2024
Merged

Remove ImplicitKeypointBBoxHead #98

merged 3 commits into from
Oct 7, 2024

Conversation

kozlov721
Copy link
Collaborator

  • Removed ImplicitKeypointBBoxHead, ImplicitKeypointBBoxLoss and KeypointLoss
  • Removed related code that is no longer used anywhere

@kozlov721 kozlov721 requested a review from a team as a code owner October 7, 2024 08:02
@kozlov721 kozlov721 requested review from klemen1999, tersekmatija and conorsim and removed request for a team October 7, 2024 08:02
@github-actions github-actions bot added the fix Fixing a bug label Oct 7, 2024
@kozlov721 kozlov721 merged commit 157b44d into dev Oct 7, 2024
5 of 6 checks passed
@kozlov721 kozlov721 deleted the fix/remove-implicit-keypoint-head branch October 7, 2024 14:47
kozlov721 added a commit that referenced this pull request Oct 9, 2024
@kozlov721
Remove ImplicitKeypointBBoxHead (#98)
e4fa3a2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@klemen1999 klemen1999 klemen1999 approved these changes

@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

@kozlov721 kozlov721

Labels
fix Fixing a bug
Projects
None yet
Milestone
No milestone
Development

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