⚡️ Speed up function calculate_shift by 8%
#127
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.
📄 8% (0.08x) speedup for
calculate_shiftinsrc/diffusers/pipelines/ltx/pipeline_ltx.py⏱️ Runtime :
639 microseconds→594 microseconds(best of457runs)📝 Explanation and details
Here is an optimized rewrite of your program for runtime and memory efficiency. (There was little room for micro-optimization in such a small, purely-arithmetic function, but every bit helps!)
mandbare computed only once for default parameters).__slots__in the helper (if a callable object is used).Note: This function is already very efficient, so changes are incremental and technical.
Summary of optimizations:
muassignment and unneeded intermediate variables.Function signature and return values remain exactly the same.
✅ Correctness verification report:
🌀 Generated Regression Tests Details
To edit these changes
git checkout codeflash/optimize-calculate_shift-mbdgxab0and push.