search

martinbiel / StochasticPrograms.jl

Julia package for formulating and analyzing stochastic recourse models.
MIT License
75 stars 25 forks source link

Maintenance of repo #51

Open avinashresearch1 opened 8 months ago

avinashresearch1 commented 8 months ago

I'm looking to build additional tooling for two-stage stochastic programs and ran across this excellent modeling package. The issue is it appears to have fallen behind on maintaining compat with the updates in JuMP, MathOptInterface and certain solvers (as other folks have pointed out).

Would you be open to transferring ownership (probably to JuMP-dev ?) or giving admin rights to new folks who can help take over maintenance? I can volunteer to maintain the package (compat etc.). Alternatively, I am considering factoring out a new repo (say called StochasticProgrammingModels.jl with files from /src/macros and updating that to fit JuMP syntax (in the spirit of having packages only doing one thing). Please let me know your thoughts.

odow commented 8 months ago

We could consider moving this to jump-dev. I don't think @martinbiel isn't active on GitHub anymore, but I'll email him.

martinbiel commented 8 months ago

Hey!

Bumping versions so that SPjl does not downgrade should not be too much of a hassle. In the long term, a more involved rewrite of the backend is required to achieve an implementation less dependent on the MOI version.

I was awaiting stable versions of JuMP/MOI before starting on that, but since then new work and fathering twins got in the way :sweat_smile:. I have added @odow as a collaborator to help out with the maintenance. In the long term, moving the repo to jump-dev is probably sensible :slightly_smiling_face:.

odow commented 8 months ago

@avinashresearch1 happy to review/merge PRs.

avinashresearch1 commented 8 months ago

Amazing, thanks @odow and @martinbiel. I have a local clone I worked on during the holiday break and wrote another example, let me clean up and send a PR. I also have some CI minutes so can run tests on my fork before it is merged.

Congrats @martinbiel on the twins :)