-
Notifications
You must be signed in to change notification settings - Fork 804
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
Various Improvements #1713
Various Improvements #1713
Conversation
bf8bc2b
to
f62805f
Compare
… as it is possible to have non-zero discrete keys
Some Updates
d552eef
to
538871a
Compare
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.
LGTM!
You can cancel that last check and merge, it takes hours. I opened a discussion on killing gcc9 altogether in CI. |
Looks like the issue is gcc9 + TBB but only with the python wrapper. I think we can just get rid of checking with gcc9+TBB but keep gcc9. |
A bunch of improvements to the code, including formatting, more tests and better printing.