⚡️ Speed up method Dinov2WithRegistersDropPath.forward by 94% in PR #1250 (feature/inference-v1-models)
#1258
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
⚡️ This pull request contains optimizations for PR #1250
If you approve this dependent PR, these changes will be merged into the original PR branch
feature/inference-v1-models.📄 94% (0.94x) speedup for
Dinov2WithRegistersDropPath.forwardininference/v1/models/rfdetr/dinov2_with_windowed_attn.py⏱️ Runtime :
16.2 milliseconds→8.36 milliseconds(best of62runs)📝 Explanation and details
Here is an optimized version of your code, focusing on runtime and memory reduction. The profiler indicates the vast majority of time is spent in the line.
We can optimize this by performing in-place operations (to reduce memory allocations and speed up computation), and by fusing more operations. Also, there is no need to construct
shapeusing Python arithmetic every call—let's use tensor broadcasting andexpand_asfor efficiency.Changes and rationale:
.div(keep_prob) * random_tensorwithinput.mul_(random_tensor).div_(keep_prob)in-place, if it is safe (as no reuse of input).expand_as(input)instead of shape tuple math.Performance rationale.
.div()followed by a*, doing only the minimum required math using fused operations.You could go further with.
But as a drop-in, this is as fast as you can get in PyTorch with the existing logic.
✅ Correctness verification report:
🌀 Generated Regression Tests Details
To edit these changes
git checkout codeflash/optimize-pr1250-2025-05-13T13.24.23and push.