Skip to content

Update dependencies #73

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Update dependencies #73

wants to merge 3 commits into from

Conversation

paraynaud
Copy link
Member

Catch up 1 year or so of dependencies

Copy link
Contributor

github-actions bot commented Jun 16, 2025

Package name latest stable
PartiallySeparableNLPModels
PartiallySeparableSolvers

@paraynaud paraynaud force-pushed the pr-update-dependencies branch 3 times, most recently from bf8ba8d to fe070a4 Compare June 16, 2025 20:14
@paraynaud paraynaud force-pushed the pr-update-dependencies branch from fe070a4 to a3f6e1b Compare June 16, 2025 20:45
@paraynaud
Copy link
Member Author

@dpo, I solved my issues with dependencies, except for nightly builds.
I checked Krylov.jl, and I saw no nightly builds, are they still a thing ?

NLPModelsJuMP = "^0"
ReverseDiff = "1"
Symbolics = "4 - 6"
julia = "1"
Copy link
Member

Choose a reason for hiding this comment

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

It's probably worth being more specific here like 1.10 or lts

Copy link
Member Author

Choose a reason for hiding this comment

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

I tried to be the more generable possible, such that julia 1.7 and julia 1.11 could pass the tests.
If I am more specific on package requirements, I get conflicts.
I know I can of trust the package manager, but it seems to work.

Just to let you know, I'll update soon my other packages.
Since I may have some tests to run from PartitionedKnetNLPModels, I'll try to keep both julia 1.7 and its latest versions running.
The idea is to ensure PartitionedKnetNLPModels will run on the setup I had at the end of the thesis (Knet didn't update anything since then, so I fear conflicts with more recent package versions).

Copy link
Member

Choose a reason for hiding this comment

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

You actually want to maintain the LTS which is Julia 1.10 and if anything below doesn't work that's fine (we no longer test it).
You can simply write julia = 1.10 and the CI should still be able to run

Copy link
Member

Choose a reason for hiding this comment

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

Obviously next release is a breaking release.

@tmigot
Copy link
Member

tmigot commented Jun 17, 2025

@dpo, I solved my issues with dependencies, except for nightly builds.
I checked Krylov.jl, and I saw no nightly builds, are they still a thing ?

If it doesn't seem to concern your package directly, you can tolerate the nighlty failing.

@paraynaud
Copy link
Member Author

If it doesn't seem to concern your package directly, you can tolerate the nighlty failing.

I get an error related to ExpressionTreeForge.jl that doesn't exist for julia 1.7 and 1.11:

WARNING: Imported binding M_interface_expr_tree._transform_to_Expr_JuMP was undeclared at import time during import to M_trait_expr_tree.
...
ERROR: LoadError: invalid method definition in M_trait_expr_tree: exported function 
M_interface_expr_tree._transform_to_Expr_JuMP does not exist

_transform_to_Expr_JuMP is the only method which is not binded.
It is a new error for me.

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.

2 participants