⚡️ Speed up function sorter by 81%
#835
Closed
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.
📄 81% (0.81x) speedup for
sorterincode_to_optimize/bubble_sort.py⏱️ Runtime :
5.83 seconds→3.22 seconds(best of5runs)📝 Explanation and details
The optimized code implements two key bubble sort optimizations that significantly improve performance:
1. Early Exit with Swap Detection: The algorithm tracks whether any swaps occurred during a pass using a
swappedflag. If no swaps happen, the array is fully sorted and the algorithm exits early. This provides massive speedups for already-sorted or nearly-sorted data (35,765% faster for sorted arrays, 36,109% faster for arrays with all duplicates).2. Reduced Comparison Range: Each pass reduces the inner loop range by
ielements (range(n - i - 1)vsrange(len(arr) - 1)), since the lastielements are guaranteed to be in their final sorted positions afteripasses. This eliminates unnecessary comparisons.3. Tuple Unpacking for Swaps: Replaces the three-line temporary variable swap with Python's tuple unpacking (
arr[j], arr[j + 1] = arr[j + 1], arr[j]), which is more efficient and eliminates one variable assignment per swap.The line profiler shows the optimized version performs ~53M inner loop iterations vs ~110M in the original (52% reduction), directly correlating to the 81% runtime improvement. The optimizations are particularly effective for:
Even for worst-case scenarios like reverse-sorted arrays, the reduced comparison range still provides 54-74% speedups.
✅ Correctness verification report:
⚙️ Existing Unit Tests and Runtime
benchmarks/test_benchmark_bubble_sort.py::test_sort2test_bubble_sort.py::test_sorttest_bubble_sort_conditional.py::test_sorttest_bubble_sort_import.py::test_sorttest_bubble_sort_in_class.py::TestSorter.test_sort_in_pytest_classtest_bubble_sort_parametrized.py::test_sort_parametrizedtest_bubble_sort_parametrized_loop.py::test_sort_loop_parametrized🌀 Generated Regression Tests and Runtime
To edit these changes
git checkout codeflash/optimize-sorter-mgzma75jand push.