[chore]: Replace tensorflow_addons by felixdittrich92 · Pull Request #1252 · mindee/doctr (original) (raw)
Navigation Menu
Appearance settings
- AI CODE CREATION
* GitHub CopilotWrite better code with AI
* GitHub Copilot appDirect agents from issue to merge
* MCP RegistryNewIntegrate external tools - DEVELOPER WORKFLOWS
* ActionsAutomate any workflow
* CodespacesInstant dev environments
* IssuesPlan and track work
* Code ReviewManage code changes - APPLICATION SECURITY
* GitHub Advanced SecurityFind and fix vulnerabilities
* Code securitySecure your code as you build
* Secret protectionStop leaks before they start - EXPLORE
* Why GitHub
* Documentation
* Blog
* Changelog
* Marketplace
- AI CODE CREATION
- BY COMPANY SIZE
* Enterprises
* Small and medium teams
* Startups
* Nonprofits - BY USE CASE
* App Modernization
* DevSecOps
* DevOps
* CI/CD
* View all use cases - BY INDUSTRY
* Healthcare
* Financial services
* Manufacturing
* Government
* View all industries
- BY COMPANY SIZE
- EXPLORE BY TOPIC
* AI
* Software Development
* DevOps
* Security
* View all topics - EXPLORE BY TYPE
* Customer stories
* Events & webinars
* Ebooks & reports
* Business insights
* GitHub Skills - SUPPORT & SERVICES
* Documentation
* Customer support
* Community forum
* Trust center
* Partners
- EXPLORE BY TOPIC
- COMMUNITY
* GitHub SponsorsFund open source developers - PROGRAMS
* Security Lab
* Maintainer Community
* Accelerator
* GitHub Stars
* Archive Program - REPOSITORIES
* Topics
* Trending
* Collections
- COMMUNITY
- Pricing
Provide feedback
We read every piece of feedback, and take your input very seriously.
Include my email address so I can be contacted
Saved searches
Use saved searches to filter your results more quickly
Appearance settings
Notifications You must be signed in to change notification settings
Additional navigation options
Merged
merged 8 commits into
mindee:mainfrom
felixdittrich92:tensorflow-addons-replacement
Jul 20, 2023
ConversationCommits (8)ChecksFiles changed
Merged
[chore]: Replace tensorflow_addons #1252
merged 8 commits into
mindee:mainfrom
felixdittrich92:tensorflow-addons-replacement
Conversation
felixdittrich92 commented
•
edited
Loading
Copy link Copy Markdown
Collaborator
This PR:
- replaces
tensorflow_addonswhich is end of life (May 2024) --> TensorFlow Addons Wind Down tensorflow/addons#2807 - add hotfix for docs build
- maybe some space for improvements
Any feedback is welcome :)
[replace rotation](/mindee/doctr/pull/1252/commits/b258f742d2ce907eefb4f437613d6d3ceaf45cd4 "replace rotation")
[b258f74](/mindee/doctr/pull/1252/commits/b258f742d2ce907eefb4f437613d6d3ceaf45cd4)
felixdittrich92 self-assigned this
felixdittrich92 added module: transforms
Related to doctr.transforms
framework: tensorflow topic: build
Related to dependencies and build
Miscellaneous
labels
felixdittrich92 added this to the 0.6.1 milestone
codecov Bot commented
•
edited
Loading
Copy link Copy Markdown
Codecov Report
Merging #1252 (732766c) into main (4e1985f) will increase coverage by
0.01%.
The diff coverage is100.00%.
@@ Coverage Diff @@ ## main #1252 +/- ##
Coverage 95.64% 95.66% +0.01%
Files 154 154
Lines 6873 6903 +30- Hits 6574 6604 +30
Misses 299 299
| Flag | Coverage Δ | |
|---|---|---|
| unittests | 95.66% <100.00%> (+0.01%) | ⬆️ |
Flags with carried forward coverage won't be shown. Click here to find out more.
| Impacted Files | Coverage Δ | |
|---|---|---|
| doctr/models/classification/vit/tensorflow.py | 100.00% <ø> (ø) | |
| doctr/models/recognition/master/tensorflow.py | 98.21% <ø> (ø) | |
| doctr/transforms/functional/tensorflow.py | 100.00% <100.00%> (ø) | |
| doctr/transforms/modules/tensorflow.py | 83.33% <100.00%> (-0.19%) | ⬇️ |
... and 2 files with indirect coverage changes
[minor update](/mindee/doctr/pull/1252/commits/67bd8690820070b7e82aa40b0ee4ad907b7b3b30 "minor update")
[67bd869](/mindee/doctr/pull/1252/commits/67bd8690820070b7e82aa40b0ee4ad907b7b3b30)
felixdittrich92 requested review fromcharlesmindee, frgfm and odulcy-mindee
felixdittrich92 added 2 commits
[trigger CI](/mindee/doctr/pull/1252/commits/406b2a5fec5d64efd592c132715f49aed9d2ca86 "trigger CI")
[406b2a5](/mindee/doctr/pull/1252/commits/406b2a5fec5d64efd592c132715f49aed9d2ca86)
[docs hotfix](/mindee/doctr/pull/1252/commits/0b7a1f430b7702c77a0b415f5ebaf3ec7d38f8d6 "docs hotfix")
[0b7a1f4](/mindee/doctr/pull/1252/commits/0b7a1f430b7702c77a0b415f5ebaf3ec7d38f8d6)
felixdittrich92 marked this pull request as ready for review
odulcy-mindee reviewed Jul 17, 2023
Comment thread
doctr/models/classification/vit/tensorflow.py Outdated
[apply suggestion](/mindee/doctr/pull/1252/commits/74681d011a4963ebff7da4ce17d243987f904790 "apply suggestion")
[74681d0](/mindee/doctr/pull/1252/commits/74681d011a4963ebff7da4ce17d243987f904790)
felixdittrich92 requested a review from odulcy-mindee
[np instead random](/mindee/doctr/pull/1252/commits/4afa59ecdf6ead011e57def5f6c9b7c32d23d11e "np instead random")
[4afa59e](/mindee/doctr/pull/1252/commits/4afa59ecdf6ead011e57def5f6c9b7c32d23d11e)
odulcy-mindee reviewed Jul 18, 2023
Comment thread
doctr/transforms/modules/tensorflow.py Outdated
felixdittrich92 added 2 commits
[apply suggestions](/mindee/doctr/pull/1252/commits/fe635af8261d20c4f7b9d4d9b509a87e63926326 "apply suggestions")
[fe635af](/mindee/doctr/pull/1252/commits/fe635af8261d20c4f7b9d4d9b509a87e63926326)
[update](/mindee/doctr/pull/1252/commits/732766ce6ffef649e44ba9403862401443f1510e "update")
[732766c](/mindee/doctr/pull/1252/commits/732766ce6ffef649e44ba9403862401443f1510e)
felixdittrich92 requested a review from odulcy-mindee
felixdittrich92 commented
Copy link Copy Markdown
Collaborator Author
@odulcy-mindee wdyt ? :)
odulcy-mindee approved these changes Jul 20, 2023
felixdittrich92 merged commit 1ad033e into mindee:main
felixdittrich92 deleted the tensorflow-addons-replacement branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
[ Show hidden characters]({{ revealButtonHref }})
Sign up for free to join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
frgfm Awaiting requested review from frgfm
charlesmindee Awaiting requested review from charlesmindee
2 more reviewers
felixT2K felixT2K left review comments
odulcy-mindee odulcy-mindee approved these changes
Reviewers whose approvals may not affect merge requirements
Assignees
Labels
Related to doctr.transforms
Related to dependencies and build
Miscellaneous
Projects
None yet
Milestone
Development
Successfully merging this pull request may close these issues.