-
Notifications
You must be signed in to change notification settings - Fork 156
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
Implemented symbolic constant E #2387
Conversation
We've tested out the constant |
Definitely add a test for E. That's to ensure that everything actually works and that it will not get broken in the future. |
Done. I've added a minimalistic test function as of now which checks overall use of |
Not really sure why the EDIT: Re-running the CI makes it pass. |
Thanks! What was the failure at the CI? |
I think so 1 test ( |
It's a flaky test then. Can you report the error into an issue with a log of the failure? We need to keep track of this, so that we can fix it. |
I've made this pull request as a helping PR for #2386