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.

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

Change PauliFrame underlying data to be UInt32 instead of Bool #173

Open
Shayan-P opened this issue Aug 26, 2023 · 0 comments
Open

Change PauliFrame underlying data to be UInt32 instead of Bool #173

Shayan-P opened this issue Aug 26, 2023 · 0 comments
Labels
GPU related to new gpu support addition

Comments

@Shayan-P
Copy link
Collaborator

Each Boolean value takes 1 byte (but we are using 1 bit of information here).

By using UInt32 and bitmask, we will improve the memory by 8x.

Also, this becomes a bottleneck when users want to create large Pauliframes. For example, with 6GB of RAM given those measurements=2000, I cannot go beyond setting trajectories to 2^20 (and it crashes there sometimes).

@Shayan-P Shayan-P added the GPU related to new gpu support addition label Aug 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GPU related to new gpu support addition
Projects
None yet
Development

No branches or pull requests

1 participant