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.

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

fix: clip hrnet scores #110

Closed
wants to merge 4 commits into from
Closed

fix: clip hrnet scores #110

wants to merge 4 commits into from

Conversation

jkbmrz
Copy link
Collaborator

@jkbmrz jkbmrz commented Oct 16, 2024

This is a quick fix for the HRNetParser. Scores with calues >1 were occurring (max. observed was 1.1) and this was causing errors later in the process.

@github-actions github-actions bot added parsers Changes affecting ml.parsers fix Fixing a bug release New version release labels Oct 16, 2024
@jkbmrz
Copy link
Collaborator Author

jkbmrz commented Oct 16, 2024

Not really sure what's wrong with the pre-commit check. Running pre-commit run --all-files locally spots no issues.

@kkeroo
Copy link
Collaborator

kkeroo commented Oct 16, 2024

PR is merging to main. Are we now switching to the concept @klemen1999 proposed where main is not the same as pypi package?

@klemen1999
Copy link
Collaborator

PR is merging to main. Are we now switching to the concept @klemen1999 proposed where main is not the same as pypi package?

Yes let's merge to main from now. But we need to be even more sure that we don't introduce breaking changes now

@klemen1999 klemen1999 removed the release New version release label Oct 16, 2024
Copy link
Collaborator

@kkeroo kkeroo left a comment

Choose a reason for hiding this comment

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

LGTM. I ran the tests locally and they went through.

@jkbmrz
Copy link
Collaborator Author

jkbmrz commented Oct 21, 2024

The changes presented here are being added to the Parsers v2 review PR #113

@jkbmrz jkbmrz closed this Oct 21, 2024
@jkbmrz jkbmrz deleted the fix/hrnet_confidence_cap branch October 24, 2024 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DevOps fix Fixing a bug parsers Changes affecting ml.parsers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants