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
Disabling bindings by default in cmake. #2782
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.
This looks good to me. Can you also update HISTORY.md
?
@Yashwants19 , I have updated |
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.
Looks good to me:+1:
Thanks for review @shrit :-D |
I am wondering why one test is now failing. Previously it was not failing. |
Some tests are non deterministic, so it's possible that they can fail some time e.g. if the threshold is to tight. |
Co-authored-by: Marcus Edel <marcus.edel@fu-berlin.de>
Okay 👍 I have fixed the style. Thanks for review :) |
Thanks @RishabhGarg108 for looking into the issue. 👍 |
This PR tries to solve #2773 .
The default options for all bindings are set to "OFF" in this.