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

Optimize _rotate_basis_state #103

Open
emerali opened this issue Dec 31, 2019 · 1 comment
Open

Optimize _rotate_basis_state #103

emerali opened this issue Dec 31, 2019 · 1 comment
Assignees
Labels
enhancement New feature or request gpu Priority: Low Size: Medium For moderately sized issues

Comments

@emerali
Copy link
Collaborator

emerali commented Dec 31, 2019

  • Create cplx-torch version of np.prod to speed up GPU run-time
  • Low prio: turn it into a generator function in order to be able to deal with a larger number of local rotations
@emerali emerali added enhancement New feature or request Size: Medium For moderately sized issues Priority: Medium gpu labels Dec 31, 2019
@emerali emerali added this to the Release v1.3 milestone Dec 31, 2019
@emerali emerali self-assigned this Dec 31, 2019
@emerali
Copy link
Collaborator Author

emerali commented Dec 31, 2019

Tried creating a cplx-torch version of np.prod, didn't actually improve GPU performance (in fact it made it slightly slower)

Implementation of cplx.prod was as so:

  • Calculate arg of every element and add them up
  • Calculate magnitude and multiply them
  • Create complex result using magnitude and arg

There may be a better way to implement it, so I'll leave this issue open for now

@emerali emerali removed this from the Release v1.3 milestone Jan 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request gpu Priority: Low Size: Medium For moderately sized issues
Projects
None yet
Development

No branches or pull requests

1 participant