Skip to content
This repository was archived by the owner on Jun 3, 2025. It is now read-only.

Conversation

@eldarkurtic
Copy link
Contributor

This PR fixes #200
(sorry for bundling auto-corrected whitespaces)

@eldarkurtic
Copy link
Contributor Author

Changes in run_qa.py are tested and should work fine.
Changes in run_distill_qa.py are not tested, but it looks like they are affected by the same miscalculation.

@markurtz markurtz merged commit 3a2b96d into neuralmagic:main Apr 29, 2021
eldarkurtic added a commit to eldarkurtic/sparseml that referenced this pull request May 6, 2021
markurtz added a commit that referenced this pull request May 12, 2021
* Update annotate.py

minor edits, thx

* Update annotate.py

Getting additional edits at top of file to match ones below

* Fix for integrations/timm checkpoint path (#198)

This PR fixes issue #197

Co-authored-by: Benjamin Fineran <bfineran@users.noreply.github.com>

* Fix steps_per_epoch calculation (#201)

Co-authored-by: Benjamin Fineran <bfineran@users.noreply.github.com>

* YOLO webcam example - add assert for webcam load (#202)

* YOLO webcam example - add assert for webcam load

* update readme to note other options for annotate

* formatting

Co-authored-by: Eldar Kurtic <eldar.ciki@gmail.com>
Co-authored-by: Benjamin Fineran <bfineran@users.noreply.github.com>
Co-authored-by: Mark Kurtz <mark@neuralmagic.com>
markurtz added a commit that referenced this pull request May 12, 2021
Updates w.r.t: #218 and #201

Co-authored-by: Mark Kurtz <mark@neuralmagic.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Number of steps per epoch in Transformers integration miscalculated

4 participants