Skip to content

Commit bb4686c

Browse files
Simplify telemetry to use GitHub.create_issue() only
- Remove complex PR/fork creation system - Replace with simple GitHub.create_issue() for SciML/LinearSolve.jl - Reduce token requirements to just "Issues: Write" permission - Remove CSV dependency and unnecessary file generation functions - Update documentation and user messaging for issue-based workflow - Use only "benchmark-data" label for created issues - Maintain local file fallback when GitHub creation fails 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent e572147 commit bb4686c

File tree

3 files changed

+40
-473
lines changed

3 files changed

+40
-473
lines changed

lib/LinearSolveAutotune/Project.toml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ version = "0.1.0"
77
LinearSolve = "7ed4a6bd-45f5-4d41-b270-4a48e9bafcae"
88
BenchmarkTools = "6e4b80f9-dd63-53aa-95a3-0cdb28fa8baf"
99
Base64 = "2a0f44e3-6c83-55bd-87e4-b1978d98bd5f"
10-
CSV = "336ed68f-0bac-5ca0-87d4-7b16caf5d00b"
1110
DataFrames = "a93c6f00-e57d-5684-b7b6-d8193f3e46c0"
1211
GitHub = "bc5e4493-9b4d-5f90-b8aa-2b2bcaad7a26"
1312
Plots = "91a5bcdd-55d7-5caf-9e0b-520d859cae80"
@@ -33,7 +32,6 @@ Metal = "dde4c033-4e86-420c-a63e-0dd931031962"
3332
LinearSolve = "3"
3433
BenchmarkTools = "1"
3534
Base64 = "1"
36-
CSV = "0.10"
3735
DataFrames = "1"
3836
GitHub = "5"
3937
Plots = "1"

lib/LinearSolveAutotune/src/LinearSolveAutotune.jl

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ module LinearSolveAutotune
22

33
using LinearSolve
44
using BenchmarkTools
5-
using CSV
65
using DataFrames
76
using PrettyTables
87
using Preferences
@@ -76,15 +75,15 @@ include("preferences.jl")
7675
Run a comprehensive benchmark of all available LU factorization methods and optionally:
7776
7877
- Create performance plots for each element type
79-
- Upload results to GitHub telemetry
78+
- Create GitHub issue with telemetry data for community collection
8079
- Set Preferences for optimal algorithm selection
8180
- Support both CPU and GPU algorithms based on hardware detection
8281
- Test algorithm compatibility with different element types
8382
8483
# Arguments
8584
8685
- `large_matrices::Bool = false`: Include larger matrix sizes for GPU benchmarking
87-
- `telemetry::Bool = true`: Share results to GitHub issue for community data
86+
- `telemetry::Bool = true`: Create GitHub issue with results for community data collection
8887
- `make_plot::Bool = true`: Generate performance plots for each element type
8988
- `set_preferences::Bool = true`: Update LinearSolve preferences with optimal algorithms
9089
- `samples::Int = 5`: Number of benchmark samples per algorithm/size
@@ -216,9 +215,9 @@ function autotune_setup(;
216215
end
217216
end
218217

219-
# Upload telemetry if requested
218+
# Create GitHub issue with telemetry if requested
220219
if telemetry && nrow(successful_results) > 0
221-
@info "📤 Preparing telemetry data for community sharing..."
220+
@info "📤 Creating GitHub issue with benchmark data for community collection..."
222221
markdown_content = format_results_for_github(results_df, system_info, categories)
223222
upload_to_github(markdown_content, plot_files, github_auth, results_df, system_info, categories)
224223
end

0 commit comments

Comments
 (0)