-
-
Notifications
You must be signed in to change notification settings - Fork 212
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
feat: add assertions
functionality
#3364
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cool :)
b7bd315
to
a571774
Compare
I think formatter fails because it needs a rebase. Will do that once the PR is approved. |
This isn't causing a NaN in the non-debug? |
I thought we didn't want this in the non-debug? I can add it if we do |
You want a NaN but not an error, or you want isoutofdomain. Either way you need step rejections |
Co-authored-by: Fredrik Bagge Carlson <baggepinnen@gmail.com>
Co-authored-by: Fredrik Bagge Carlson <baggepinnen@gmail.com>
add logging in `debug_system`
a571774
to
b968b92
Compare
Checklist
contributor guidelines, in particular the SciML Style Guide and
COLPRAC.
Additional context
Add any other context about the problem here.