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

[ADNLPModels] Upgrade to v0.8.2 #65

Merged
merged 1 commit into from
Jun 23, 2024

Conversation

amontoison
Copy link
Contributor

@amontoison amontoison commented Jun 22, 2024

  • We replaced Symbolics.jl with SparseConnectivityTracer.jl for detecting the sparsity pattern of Jacobians and Hessians.
  • We also replaced ColPack.jl with SparseMatrixColorings.jl for coloring sparse Jacobians and Hessians.

@ccoffrin
Copy link
Member

Thanks! Merging into a branch for testing.

@ccoffrin ccoffrin merged commit fdeb482 into lanl-ansi:maintenance-2406 Jun 23, 2024
5 checks passed
@amontoison amontoison deleted the adnlpmodels-upgrade branch June 23, 2024 16:09
ccoffrin added a commit that referenced this pull request Jun 23, 2024
* [ADNLPModels] Upgrade to v0.8.2 (#65)
* update all deps to latest versions

---------

Co-authored-by: Alexis Montoison <[email protected]>
@gdalle
Copy link

gdalle commented Jun 24, 2024

We replaced Symbolics.jl with SparsityDetector.jl for detecting the sparsity pattern of Jacobians and Hessians.

The correct package name is SparseConnectivityTracer.jl

@ccoffrin
Copy link
Member

Just to confirm, nothing needs to be revised in the experiment I just ran. This is just a correction to the notes in this PR. Is that correct?

@amontoison
Copy link
Contributor Author

Yes, I just did a typo in the name of the package.
Sorry @gdalle!

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.

3 participants