Conversation
💊 CI failures summary and remediationsAs of commit 01be6a5 (more details on the Dr. CI page): 💚 💚 Looks good so far! There are no failures yet. 💚 💚 This comment was automatically generated by Dr. CI (expand for details).Please report bugs/suggestions to the (internal) Dr. CI Users group. |
9214973 to
17593a5
Compare
db77373 to
1221e12
Compare
NicolasHug
left a comment
There was a problem hiding this comment.
Thanks @datumbox , I took a brief look only. Minor question but otherwise LGTM
| eager_out = nn_module(*args) | ||
| if eager_out is None: | ||
| with torch.no_grad(), freeze_rng_state(): | ||
| if unwrapper: |
There was a problem hiding this comment.
Is this line needed? It looks like it eager_out wasn't unwrapped before
There was a problem hiding this comment.
It's not mandatory to have it for most of the existing models but this is only due to implementation details. Since this is a general purpose tool for checking JIT-scriptability, I opted for consistently unwrapping the output in all place.
FYI the reason many models don't have to get unwrapped is due to idioms like this:
vision/torchvision/models/googlenet.py
Lines 163 to 179 in 4ae20e5
New non-detection models don't use this idiom any more (returning different output depending on jit/training flag), so I think it's safer to handle it explicitly.
jdsgomes
left a comment
There was a problem hiding this comment.
LGTM! Thanks for this change - great improvement!
|
Hey @datumbox! You merged this PR, but no labels were added. The list of valid labels is available at https://github.com/pytorch/vision/blob/main/.github/process_commit.py |
Summary: * Measuring execution times of models. * Speed up models by avoiding re-estimation of eager output * Fixing linter * Reduce input size for big models * Speed up jit check method. * Add simple jitscript fallback check for flaky models. * Restore pytest filtering * Fixing linter Reviewed By: vmoens Differential Revision: D34878998 fbshipit-source-id: 37bfa05aac0d28d59d3320119147446006bff75c
We focus on the
test_classification_model,test_detection_model,test_quantized_classification_model,test_segmentation_model,test_video_modeltests and improve their execution times by 20%.This is achieved by:
Before: 629.21 sec
Run: https://app.circleci.com/pipelines/github/pytorch/vision/15497/workflows/51d7e30b-86ff-4c71-af68-7cb3438f25c9/jobs/1252313
After: 507.28 sec
Run: https://app.circleci.com/pipelines/github/pytorch/vision/15520/workflows/2cc25bdc-fe10-4705-b942-c206a9d12fad/jobs/1254266