Skip to content

Conversation

@francis-biocurie
Copy link
Contributor

@francis-biocurie francis-biocurie commented Sep 8, 2025

Checklist

  • Appropriate tests were added
  • Any code changes were done in a way that does not break public API
  • All documentation related to code changes were updated
  • The new code follows the
    contributor guidelines, in particular the SciML Style Guide and
    COLPRAC.
  • Any new documentation only uses public API

Additional context

I made the two following changes:

  1. The i argument was not used, so the initial guesses were all identical.
  2. I created two seperated markdown blocks, to be able to see the objective value of the first non-multistart optimization in the online documentation.

Also: maybe we should explain what is the repeat argument here ? I don't understand its role. I'm not able to find any explanation about this argument in the SciML documentation.

@ChrisRackauckas ChrisRackauckas merged commit 4f072a7 into SciML:master Sep 9, 2025
@ChrisRackauckas
Copy link
Member

Also: maybe we should explain what is the repeat argument here ? I don't understand its role. I'm not able to find any explanation about this argument in the SciML documentation.

If the solution was rejected, then repeat=true so you know it's a repeat solve. It's something that is mostly useful for stochastic algorithms, like SDEs, which might fail on some noise instantiations. Stochastic optimizers could be a use case here: if you have a stochastic optimizer diverge, you can reject that from your ensemble and have it just run those again.

@francis-biocurie francis-biocurie deleted the patch-1 branch September 9, 2025 12:37
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