revise the created pauli_term_grouping.py #1100
Conversation
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| """ | ||
|
|
||
| import numpy as np | ||
| from typing import List, Dict, Set, Tuple, Optional, Union |
Check notice
Code scanning / CodeQL
Unused import Note
| from typing import List, Dict, Set, Tuple, Optional, Union | ||
| import networkx as nx | ||
| from collections import defaultdict, Counter | ||
| from itertools import combinations |
Check notice
Code scanning / CodeQL
Unused import Note
|
|
||
| try: | ||
| from openfermion.ops import QubitOperator | ||
| from openfermion.utils import commutator, count_qubits |
Check notice
Code scanning / CodeQL
Unused import Note
| # Apply physical constraints | ||
| n_clusters = max(2, min(n_clusters, self.n_terms // 3)) | ||
|
|
||
| except: |
Check notice
Code scanning / CodeQL
Except block handles 'BaseException' Note
| """ | ||
| # Compute pairwise distances | ||
| distances = pdist(embedding, metric='euclidean') | ||
| distance_matrix = squareform(distances) |
Check notice
Code scanning / CodeQL
Unused local variable Note
| best_score = score | ||
| best_n_clusters = n_clusters | ||
|
|
||
| except: |
Check notice
Code scanning / CodeQL
Except block handles 'BaseException' Note
|
|
||
| if __name__ == "__main__": | ||
| # Run comprehensive demonstration | ||
| demo_results = demonstrate_advanced_pauli_grouping() |
Check notice
Code scanning / CodeQL
Unused global variable Note
|
@hofong428 thank you for this work. It looks like Google lacks evidence of a Contributor License Agreement (CLA) having been accepted – could you check the instructions and do that? (It won't ask you to give up any rights.) Also, could you provide a somewhat more informative title for this PR? It looks like the revision has to do with optimizing performance, so perhaps it should mention that. This will make it easier for people to understand what's going on in the future. |
|
I've tried to conduct the PRs as codebatai(my startup company), or is only
personal PR allowable?
…On Sat, Aug 2, 2025 at 6:59 AM Michael Hucka ***@***.***> wrote:
*mhucka* left a comment (quantumlib/OpenFermion#1100)
<#1100 (comment)>
@hofong428 <https://github.com/hofong428> thank you for this work. It
looks like Google lacks evidence of a Contributor License Agreement (CLA)
having been accepted – could you check the instructions and do that? (It
won't ask you to give up any rights.)
Also, could you provide a somewhat more informative title for this PR? It
looks like the revision has to do with optimizing performance, so perhaps
it should mention that. This will make it easier for people to understand
what's going on in the future.
—
Reply to this email directly, view it on GitHub
<#1100 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AQ34Y4K57MWWEJ7EPZTMWJT3LPWMDAVCNFSM6AAAAACCUVQTDCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTCNBVHE2TINRXGA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Yes, it is normally possible for an employee of a company to submit contributions too. It is up to the company to set guidelines for what their employees can do. In either case, a CLA is still needed by Google. The comment by the CLA bot above includes a link to more information. The gist of it is that you visit https://cla.developers.google.com/. Near the bottom of that page, there should be two buttons: one for "Only yourself" and another for "Your employer". To understand the implications of the choices, it's best to read the info on the "About" page mentioned below the buttons. (Here's a direct link: https://cla.developers.google.com/about) The "About" page (the page describing CLAs) is a bit long, but hopefully it will not be alarming. |
Optimized Pauli term grouping strategies for measurement reduction in fermionic simulations.
Based on research showing up to 50% measurement reduction compared to traditional techniques.