-
Notifications
You must be signed in to change notification settings - Fork 127
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
Unsupported atoms are simply ignored? #616
Comments
An exception should be raised on such cases. |
Thanks! We will take care of this issue soon! |
@UnixJunkie totally agree. I closed this issue only because it has not been acknowledged for over a year and I don't even know if it's still relevant. Thanks @yueyericardo for re-opening an looking into this. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I would expect the following code to fail
but it works without error/warnings despite the unsupported atom
Is this intended? I understand the need to ignore atoms internally due to padding, but ignoring input atoms silently feels very error-prone to me (from an user perspective).
The text was updated successfully, but these errors were encountered: