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
Update tapering.py
to support operator arithmetic
#4252
Conversation
Hello. You may have forgotten to update the changelog!
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Jaybsoni! Leaving an initial review will leave more comments after testing. Currently, tests for HF tapering and Observable tapering are missing. Overall looking good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Jaybsoni, left some small comments and questions. My main points are:
- Have we change the type of input arguments in any user-facing function?
- There seems to be some inconsistency in defining the input types in the docstrings. For instance I see both
generators (list[Union[~.Hamiltonian, ~.Operator]])
andgenerators (list[~.Operator])
. These should be corrected if they are not.
Co-authored-by: soranjh <40344468+soranjh@users.noreply.github.com>
Codecov Report
@@ Coverage Diff @@
## master #4252 +/- ##
=======================================
Coverage 99.79% 99.79%
=======================================
Files 352 352
Lines 31958 31982 +24
=======================================
+ Hits 31891 31915 +24
Misses 67 67
|
Co-authored-by: Utkarsh <utkarshazad98@gmail.com>
[sc-38623] |
* save changes to tapering * add tapering update + tests * more testing * lint and tests * more tests * Apply suggestions from code review Co-authored-by: soranjh <40344468+soranjh@users.noreply.github.com> * added more tests * fix bug * lint * lint * lint * lint + codecov * more tests * Added test and warning message for coverage * Update pennylane/ops/functions/generator.py Co-authored-by: Utkarsh <utkarshazad98@gmail.com> * lint * lint and codefactor * lint + changelog * fix tests + lint * code review --------- Co-authored-by: soranjh <40344468+soranjh@users.noreply.github.com> Co-authored-by: Utkarsh <utkarshazad98@gmail.com>
No description provided.