Skip to content
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.

Sign up for GitHub

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

Jump to bottom

ZeroState vs Allocate #513

Open
mpharrigan opened this issue Nov 13, 2023 · 1 comment · Fixed by #619
Open

ZeroState vs Allocate #513

mpharrigan opened this issue Nov 13, 2023 · 1 comment · Fixed by #619
Labels
bloqs/basic_gates

Comments

@mpharrigan
Copy link
Collaborator

These two things result in confusion. Especially since there's an unused/unsupported n attribute in ZeroState. I suggest removing this and making ZeroState always a one-bit thing for use in smaller circuits and bit twiddling. Maybe rename to ZeroBitState? That's pretty long though. If you just need a bunch of workspace use Allocate or its convenience method bb.alloc(). If you need a particular integer there's IntState.
This was referenced Jan 31, 2024
Closed
Merged
Closed
Merged
@mpharrigan
Copy link
Collaborator Author

Un-closing. We should remove n from ZeroState.

@mpharrigan mpharrigan reopened this Feb 2, 2024
@mpharrigan mpharrigan modified the milestone: After v1.0 Aug 8, 2024
@mpharrigan mpharrigan mentioned this issue Aug 19, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bloqs/basic_gates
Projects
None yet
Milestone
No milestone
1 participant
@mpharrigan