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

Analytical Pauli Coefficients #78

Draft
wants to merge 21 commits into
base: main
Choose a base branch
from
Draft

Conversation

majafranz
Copy link
Collaborator

@majafranz majafranz commented Jan 24, 2025

Do not merge yet!

This PR adds the analytical computation of the Fourier coefficients by the method of Nemkov et al., which was later extended by Wiedmann et al..

The implementation is also inspired by the corresponding code for Nemkov et al.'s paper.

TODOs:

  • Implement Pauli-Clifford circuit (Commute all clifford gates to the end, and integrate in the variable.)
  • Tree algorithm for sine-cosine sum representation
  • Sine-cosine sum to Fourier series
  • Fourier series to spectrum and coefficients
  • Documentation

@pep8speaks
Copy link

pep8speaks commented Jan 24, 2025

Hello @majafranz! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 56:35: E203 whitespace before ':'
Line 57:44: E203 whitespace before ':'

Line 342:28: E261 at least two spaces before inline comment

Line 814:89: E501 line too long (164 > 88 characters)

Comment last updated at 2025-01-24 14:38:54 UTC

@stroblme stroblme marked this pull request as draft February 3, 2025 12:46
@stroblme stroblme changed the title [Draft] Analytical Pauli Coefficients Analytical Pauli Coefficients Feb 3, 2025
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