Skip to content

Re-add PEtab.jl tutorial to docs#1455

Merged
TorkelE merged 3 commits intoSciML:masterfrom
sebapersson:petab
Apr 7, 2026
Merged

Re-add PEtab.jl tutorial to docs#1455
TorkelE merged 3 commits intoSciML:masterfrom
sebapersson:petab

Conversation

@sebapersson
Copy link
Copy Markdown
Contributor

This PR re-adds PEtab.jl to the docs (with PEtab.jl v5 syntax). In addition to adding PEtab.jl back two changes where made:

  1. I have made sure the entire docs is now compatible with Optim.jl v2 by bumping the compat in a subset of DynamicalSystems packages, and this also resulted in
  2. OptimizationEvolutionary being removed. The package does not seem to be maintained (last commit 3 years ago), and it blocked updating the docs to v2.

@sebapersson
Copy link
Copy Markdown
Contributor Author

sebapersson commented Apr 6, 2026

I am unsure why the CI fails, as ChaosTools.jl has support for Optim v2. Even more confusing, I can resolve the environment locally, and before I added Optim to the toml the environment resolved. Any ideas?

@TorkelE TorkelE closed this Apr 7, 2026
@TorkelE TorkelE reopened this Apr 7, 2026
sol = solve(optprob, Evolutionary.GA())
nothing # hide
```
to solve `optprob` for this combination of solve and implementation.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The idea here was to show off how different optimisation solvers could be used. However, I agree that it makes sense to not use an optimisation sub-library that do not see support anymore. Happy to merge like this and I will have a look later to see if there is another optimisation sublibrary and solver I can add in to this example instead.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks, I think the text will work as it currently is, and re-adding another optimizer down the line sounds good.

@TorkelE TorkelE merged commit 74b143b into SciML:master Apr 7, 2026
2 of 3 checks passed
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