Skip to content

replace reflect_about_zero in walk operators in notebooks#1496

Merged
orsa-classiq merged 2 commits intoClassiq:mainfrom
orsa-classiq:update_walk_operators
Feb 24, 2026
Merged

replace reflect_about_zero in walk operators in notebooks#1496
orsa-classiq merged 2 commits intoClassiq:mainfrom
orsa-classiq:update_walk_operators

Conversation

@orsa-classiq
Copy link
Collaborator

No description provided.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@@ -359,12 +359,17 @@
"\n",
Copy link
Member

@TomerGoldfriend TomerGoldfriend Feb 24, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line #18.                    lambda: control(state.block == 0, lambda: phase(pi)),

Here you could put my_reflect_about_zero no?


Reply via ReviewNB

@@ -78,7 +78,7 @@
"name": "stdout",
Copy link
Member

@TomerGoldfriend TomerGoldfriend Feb 24, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line #13.    print(classiq.__version__)

I see it was here before, but maybe remove this?


Reply via ReviewNB

Copy link
Member

@TomerGoldfriend TomerGoldfriend left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I left some minor comments.

@orsa-classiq orsa-classiq merged commit e20a3ff into Classiq:main Feb 24, 2026
3 checks passed
@github-actions
Copy link

🏆 Incredible, @orsa-classiq! You've merged your 81th PR! 🎯🎊

Your ongoing commitment to classiq-library is truly remarkable. You're a driving force in our community! 🚀
Your contributions are helping to shape the future of quantum computing! What exciting features or improvements do you envision next? 🔮

We are grateful for your dedication! 💫

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants