Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Simplify precompilation options #700

Merged
merged 3 commits into from
Feb 10, 2025
Merged

Simplify precompilation options #700

merged 3 commits into from
Feb 10, 2025

Conversation

willow-ahrens
Copy link
Collaborator

PrecompileTools now recognizes "precompile_workload" preference.

src/Finch.jl Outdated
Comment on lines 325 to 327
for j in _, i in _
y[i] += A[i, j] * x[j]
end
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[JuliaFormatter] reported by reviewdog 🐶

Suggested change
for j in _, i in _
y[i] += A[i, j] * x[j]
end
for j in _, i in _
y[i] += A[i, j] * x[j]

src/Finch.jl Outdated
y[i] += A[i, j] * x[j]
end
end
))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[JuliaFormatter] reported by reviewdog 🐶

Suggested change
))
end
))

src/Finch.jl Outdated

formats = []
Ts = [Int, Float64]#, Bool]

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

src/Finch.jl Outdated
if @load_preference("precompile", true)
@info "Running enhanced precompilation... (to disable, run `using Preferences; Preferences.set_preferences!(\"Finch\", \"precompile\"=>false)`"
include("../test/precompile.jl")

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

src/Finch.jl Outdated
end
A = deepcopy(format)
B = deepcopy(format)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

src/Finch.jl Outdated
end

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

Copy link

codecov bot commented Feb 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Files with missing lines Coverage Δ
src/Finch.jl 94.68% <100.00%> (+4.88%) ⬆️

@willow-ahrens willow-ahrens merged commit 2e89d22 into main Feb 10, 2025
8 checks passed
@willow-ahrens willow-ahrens deleted the wma/update-precompile branch February 10, 2025 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant