[CI] change old cache action and skip TF classification onnx export temporarily by felixT2K · Pull Request #1201 · mindee/doctr (original) (raw)
Conversation
This PR:
- update last missing cache action
- remove fixed opset version which are not more needed after changing the minimum pt / tf version)
- temporarily skip on CI for TF classification onnx export (tf2onnx 1.14 seems to raise a memory leak which kills the CI machine) - locally all tests passes for me, but yeah the tf resnet >= 31 now takes ~16GB RAM while converting
Not sure if it comes from TF itself or tf2onnx
Maybe setting up a larger runner: @charlesmindee @odulcy-mindee (or we skip more tf onnx tests ..ftm)
https://docs.github.com/en/actions/using-github-hosted-runners/using-larger-runners
Any feedback is welcome 🤗
[CI] change old cache action [CI] change old cache action and skip TF classification onnx export temporarily
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks Felix !
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 }})