Skip to content
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

Consider using ArgCheck.jl #57

Open
cgarling opened this issue Jan 5, 2025 · 0 comments
Open

Consider using ArgCheck.jl #57

cgarling opened this issue Jan 5, 2025 · 0 comments

Comments

@cgarling
Copy link
Owner

cgarling commented Jan 5, 2025

Technically, @assert calls are not guaranteed to be run depending on the optimization level that Julia is run at, but in practice right now they always do. In the future this might not be the case. It is recommended that checks on function arguments / user data be done with error throws (e.g., if x < 0; throw(DomainError ...); end) or an alternative implementation like ArgCheck.jl that is always guaranteed to run.

I don't care to fix this right now but it is worth keeping an eye on in future releases.

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

No branches or pull requests

1 participant