Commit fb35143
Improve LinearSolveAutotune UI/UX (#686)
* Improve LinearSolveAutotune UI/UX
- Remove all token authentication code from main autotune flow
- Split autotuning and result sharing into separate functions
- Add flexible size categories (small/medium/large/big) replacing binary large_matrices flag
- Add clear gh CLI setup instructions in README
- Make telemetry opt-in via explicit share_results() call
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <[email protected]>
* Enhance UI/UX with progress bars and AutotuneResults object
- Add progress bar showing algorithm being benchmarked with percentage
- Adjust size ranges: medium now goes to 300, large is 300-1000
- Create AutotuneResults struct with nice display output
- Add plot() method for AutotuneResults to create composite plots
- Update default to include large matrices (small, medium, large)
- Add clear call-to-action in results display for sharing
- Add ProgressMeter dependency
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <[email protected]>
* Fix ProgressMeter usage and delay plot generation
- Fix ProgressMeter.update\! to use 'desc' parameter instead of 'description'
- Remove make_plot parameter from autotune_setup
- Move plot generation from autotune_setup to plot(results) method
- Remove plot uploading from GitHub sharing (plots not shared anymore)
- Simplify AutotuneResults struct to only contain results_df and sysinfo
- Update documentation to reflect on-demand plot generation
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <[email protected]>
* Reorganize size categories and improve preference handling
- Add 'tiny' category (5-20), reorganize ranges: small (20-100), medium (100-300), large (300-1000)
- Change default to benchmark tiny/small/medium/large (no big) with Float64 only
- Implement intelligent type fallback for preferences:
- Float32 uses Float64 if not benchmarked
- ComplexF32 uses Float64 if not benchmarked
- ComplexF64 uses ComplexF32 then Float64 if not benchmarked
- Handle RFLU special case for complex numbers (avoids if alternative within 20% performance)
- Update preference keys to use eltype_sizecategory format (e.g., Float64_tiny)
- Set preferences for all 4 types across all 5 size categories
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <[email protected]>
* Fix sysinfo type handling and add comprehensive benchmark suggestion
- Fix AutotuneResults to properly handle sysinfo as Dict (convert from DataFrame)
- Add suggestion in display output for running comprehensive benchmarks
- Show script for testing all sizes and element types in results display
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <[email protected]>
* Update tests for new API
- Update tests to use new size categories (tiny, small, medium, large)
- Update tests for AutotuneResults type instead of tuple return
- Update preference management tests for new key format
- Remove deprecated large_matrices parameter from tests
- Add tests for AutotuneResults display method
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <[email protected]>
* Add CPUSummary.jl integration and final UI improvements
- Added CPUSummary.jl dependency for better system info
- Exported plot function for AutotuneResults
- Reordered display output: comprehensive first, community second, share last
- Updated system info gathering to use CPUSummary functions
- Enhanced OS and thread information display
* Fix CPUSummary integration - use correct function names
- Fixed CPUSummary.num_cores() instead of num_physical_cores()
- Use Sys.CPU_THREADS for logical cores
- Use Threads.nthreads() for Julia threads
- Fixed BLAS thread count with LinearAlgebra.BLAS.get_num_threads()
- Use standard Julia functions where CPUSummary doesn't provide equivalents
* Fix CPUSummary type conversion for num_cores
- Convert Static.StaticInt to regular Int for compatibility
- Ensures tests pass with CPUSummary.num_cores() output
* Fix telemetry system info key compatibility
- Use get() with fallbacks for all system info fields
- Handle both get_system_info() and get_detailed_system_info() key names
- Support both old and new key formats for compatibility
---------
Co-authored-by: ChrisRackauckas <[email protected]>
Co-authored-by: Claude <[email protected]>1 parent fd44500 commit fb35143
File tree
8 files changed
+757
-324
lines changed- lib/LinearSolveAutotune
- src
- test
8 files changed
+757
-324
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
| 10 | + | |
10 | 11 | | |
11 | 12 | | |
12 | 13 | | |
| |||
18 | 19 | | |
19 | 20 | | |
20 | 21 | | |
| 22 | + | |
21 | 23 | | |
22 | 24 | | |
23 | 25 | | |
| |||
28 | 30 | | |
29 | 31 | | |
30 | 32 | | |
| 33 | + | |
31 | 34 | | |
32 | 35 | | |
33 | 36 | | |
| |||
39 | 42 | | |
40 | 43 | | |
41 | 44 | | |
| 45 | + | |
42 | 46 | | |
43 | 47 | | |
44 | 48 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
| 125 | + | |
| 126 | + | |
| 127 | + | |
| 128 | + | |
| 129 | + | |
| 130 | + | |
| 131 | + | |
| 132 | + | |
| 133 | + | |
| 134 | + | |
| 135 | + | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
| 147 | + | |
| 148 | + | |
| 149 | + | |
| 150 | + | |
| 151 | + | |
| 152 | + | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
| 159 | + | |
| 160 | + | |
| 161 | + | |
| 162 | + | |
| 163 | + | |
| 164 | + | |
| 165 | + | |
| 166 | + | |
| 167 | + | |
| 168 | + | |
| 169 | + | |
| 170 | + | |
| 171 | + | |
| 172 | + | |
| 173 | + | |
0 commit comments