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] TF - add bf16 numpy dtype conversion #1344

Closed

Conversation

felixdittrich92
Copy link
Contributor

ref.: #1342

@felixdittrich92 felixdittrich92 added type: bug Something isn't working module: models Related to doctr.models framework: tensorflow Related to TensorFlow backend topic: text detection Related to the task of text detection topic: text recognition Related to the task of text recognition labels Oct 11, 2023
@felixdittrich92 felixdittrich92 added this to the 0.7.1 milestone Oct 11, 2023
@felixdittrich92 felixdittrich92 self-assigned this Oct 11, 2023
@codecov
Copy link

codecov bot commented Oct 11, 2023

Codecov Report

Merging #1344 (e942c16) into main (50d65d7) will decrease coverage by 0.01%.
Report is 1 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1344      +/-   ##
==========================================
- Coverage   95.78%   95.77%   -0.01%     
==========================================
  Files         154      154              
  Lines        6902     6910       +8     
==========================================
+ Hits         6611     6618       +7     
- Misses        291      292       +1     
Flag Coverage Δ
unittests 95.77% <100.00%> (-0.01%) ⬇️

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

Files Coverage Δ
...s/detection/differentiable_binarization/pytorch.py 97.77% <100.00%> (ø)
...etection/differentiable_binarization/tensorflow.py 95.16% <100.00%> (ø)
doctr/models/detection/linknet/pytorch.py 98.19% <100.00%> (ø)
doctr/models/detection/linknet/tensorflow.py 94.89% <100.00%> (ø)
doctr/models/modules/transformer/pytorch.py 100.00% <100.00%> (ø)
doctr/models/recognition/crnn/pytorch.py 94.68% <100.00%> (ø)
doctr/models/recognition/crnn/tensorflow.py 97.59% <100.00%> (ø)
doctr/models/recognition/master/pytorch.py 95.53% <100.00%> (+0.04%) ⬆️
doctr/models/recognition/master/tensorflow.py 98.23% <100.00%> (+0.01%) ⬆️
doctr/models/recognition/parseq/pytorch.py 97.18% <100.00%> (+0.01%) ⬆️
... and 7 more

... and 2 files with indirect coverage changes

chunyuan-w added a commit to chunyuan-w/doctr that referenced this pull request Oct 12, 2023
@felixdittrich92 felixdittrich92 removed this from the 0.7.1 milestone Oct 12, 2023
@felixdittrich92
Copy link
Contributor Author

overwritten by #1346

@felixdittrich92 felixdittrich92 deleted the tf-dtype-bf16-fix branch August 30, 2024 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
framework: tensorflow Related to TensorFlow backend module: models Related to doctr.models topic: text detection Related to the task of text detection topic: text recognition Related to the task of text recognition type: bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant