-
Notifications
You must be signed in to change notification settings - Fork 208
Minor corrections to new period finding demo #1357
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Conversation
👋 Hey, looks like you've updated some demos! 🐘 Don't forget to update the Please hide this comment once the field(s) are updated. Thanks! |
@alvaro-at-xanadu do we update the modification date if it's just a few typos? |
Thank you for opening this pull request. You can find the built site at this link. Deployment Info:
Note: It may take several minutes for updates to this pull request to be reflected on the deployed site. |
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.
Nice, thanks @mariaschuld 🙏 💯
Now I actually understood why there are seven qubits, I got it wrong before!
Up to the question about modification date, it looks good to me 👍
Hi @mariaschuld Yes, this is the best practice even if corrections are minor |
Fixes some minor issues raised by readers.