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

[Testing] Expand embedding tests for all operators #272

Open
jpmoutinho opened this issue Aug 30, 2024 · 1 comment
Open

[Testing] Expand embedding tests for all operators #272

jpmoutinho opened this issue Aug 30, 2024 · 1 comment
Assignees
@vytautas-a
Labels
feature New feature or request testing

Comments

@jpmoutinho
Copy link
Collaborator

Describe the feature

We should have more comprehensive tests for correctness of the Embedding class.

The tests should include:

  • Testing every gate
  • Testing every composite gate (Sequence, Add, Scale)
  • Testing the run method (the sample and expectation use the run anyway)
  • Testing the expectation method for an observable with embedded parameters
  • Testing the tensor method

It should be implemented because

No response

Additional context

No response

Would you like to work on this issue?

Yes
@jpmoutinho jpmoutinho added feature New feature or request testing labels Aug 30, 2024
@mlahariya
Copy link
Collaborator

@mlahariya - add this to the mega test refactor issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vytautas-a vytautas-a

Labels
feature New feature or request testing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mlahariya @jpmoutinho @vytautas-a