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] Fix mypy errors #1419

Merged
merged 2 commits into from
Jan 4, 2024
Merged

Conversation

felixdittrich92
Copy link
Contributor

@felixdittrich92 felixdittrich92 commented Jan 3, 2024

This PR:

@odulcy-mindee I checked the function calls and everything is fine (only switch from int to float in fillPoly)
Any feedback is welcome :)

PS: I think we can remove the ignores after the next opencv / mypy patch release

Copy link

codecov bot commented Jan 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (ed927bd) 95.79% compared to head (7e7c00c) 95.78%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1419      +/-   ##
==========================================
- Coverage   95.79%   95.78%   -0.02%     
==========================================
  Files         155      155              
  Lines        6950     6950              
==========================================
- Hits         6658     6657       -1     
- Misses        292      293       +1     
Flag Coverage Δ
unittests 95.78% <100.00%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@felixdittrich92 felixdittrich92 changed the title [CI] increase CI style version to latest secure py version [FIX] Fix mypy errors Jan 3, 2024
@felixdittrich92 felixdittrich92 marked this pull request as ready for review January 3, 2024 10:53
@felixdittrich92 felixdittrich92 self-assigned this Jan 3, 2024
@felixdittrich92 felixdittrich92 added type: bug Something isn't working topic: ci Related to CI type: code quality Related to code quality labels Jan 3, 2024
@felixdittrich92 felixdittrich92 added this to the 0.8.0 milestone Jan 3, 2024
Copy link
Collaborator

@odulcy-mindee odulcy-mindee left a comment

Choose a reason for hiding this comment

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

Thanks Felix for this fix !

@felixdittrich92 felixdittrich92 merged commit 7cc10d2 into mindee:main Jan 4, 2024
66 of 67 checks passed
@felixdittrich92 felixdittrich92 deleted the mypy-fixes branch January 4, 2024 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: ci Related to CI type: bug Something isn't working type: code quality Related to code quality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants