-
Notifications
You must be signed in to change notification settings - Fork 848
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
Product/matanvax2/cad 25786 remove prepare int #687
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
9b5ee83
to
39f5456
Compare
@@ -83,11 +83,11 @@ | |||
" N_modulus: CInt,\n", |
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.
Why did you add the preferences? do you know if the optimization level affects the results?
Reply via ReviewNB
f9f6f96
to
c2e99f5
Compare
🚂 Fantastic work @matanvax2! Your very first PR to classiq-library has been merged! 🎉🥳 If you're feeling adventurous, why not dive into another issue and keep contributing? The Classiq community would love to see more from you! 🚀 Happy coding! 👩💻👨💻 |
PR Description
Functions
prepare_int
andinplace_prepare_int
are going to be deprecated in the coming release. This PR eliminates their use in the library.