Skip to content

Make CheckInit the default for ODEs and DAEs: breaking #2602

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 1 commit into
base: master
Choose a base branch
from

Conversation

Ickaser
Copy link
Contributor

@Ickaser Ickaser commented Feb 14, 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

Context

Bulk of the discussion happened in #2514 , see particularly this comment . This is a separate pull request because I did a clumsy job rebasing that branch and filled the Git history with all the commits since November.

#2554 worked on this in a non-breaking way, including exporting BrownFullBasicInit and ShampineCollocationInit from ODENonlinearSolve.

Changes

Currently this is a very simple pull request.

This will require some documentation, which I haven't done yet.
Note to self: probably needs it at

@ChrisRackauckas
Copy link
Member

I think this looks good for v7

@Ickaser Ickaser mentioned this pull request Apr 29, 2025
9 tasks
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