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

Prepare for AdjointFactorization #13

Merged
merged 2 commits into from
Aug 23, 2023
Merged

Conversation

dkarrasch
Copy link
Member

This prepares the ground for JuliaLang/julia#46874.

@codecov-commenter
Copy link

codecov-commenter commented Jan 11, 2023

Codecov Report

Merging #13 (3b3b487) into main (70ebf7b) will not change coverage.
The diff coverage is 100.00%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##             main      #13   +/-   ##
=======================================
  Coverage   79.66%   79.66%           
=======================================
  Files           1        1           
  Lines         300      300           
=======================================
  Hits          239      239           
  Misses         61       61           
Impacted Files Coverage Δ
src/KLU.jl 79.66% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@dkarrasch
Copy link
Member Author

Gentle bump.

@dkarrasch
Copy link
Member Author

This is ready to go. It passes tests on my PR branch.

@dkarrasch
Copy link
Member Author

Should I just merge and release myself? Without these changes, this packages is broken on Julia v1.10+.

@Wimmerer @ViralBShah

@ViralBShah
Copy link
Member

Yes, please go ahead.

@rayegun
Copy link
Member

rayegun commented Aug 23, 2023

I didn't even see these, please go ahead. Thanks @dkarrasch

@dkarrasch
Copy link
Member Author

Locally, tests fail on v1.10-beta1, but that looks unrelated. I'll merge this and then we take it from there.

@dkarrasch dkarrasch merged commit e3ff1fb into JuliaSparse:main Aug 23, 2023
@dkarrasch dkarrasch deleted the dk/fact branch August 23, 2023 16:02
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.

4 participants