Finish removal of problematic line

Remove the commented out lines related to the (erroneous) ComplementPOVMEffect check. Also clean up a few misc commented out bits of code.

Author: Corey Ostrove

pygsti/algorithms/fiducialselection.py

@@ -712,7 +712,6 @@ def create_meas_cache(model, available_meas_fid_list, circuit_cache=None):
     if circuit_cache is not None:
         for povm in model.povms.values():
             for E in povm.values():
-                #if isinstance(E, _ComplementPOVMEffect): continue  # complement is dependent on others
                 new_povm_effect_key_pair= (povm.to_vector().tobytes(), E.to_dense().tobytes())
                 keypairlist.append(new_povm_effect_key_pair)
                 for measFid in available_meas_fid_list:
@@ -721,7 +720,6 @@ def create_meas_cache(model, available_meas_fid_list, circuit_cache=None):
     else:
         for povm in model.povms.values():
             for E in povm.values():
-                #if isinstance(E, _ComplementPOVMEffect): continue  # complement is dependent on others
                 new_povm_effect_key_pair= (povm.to_vector().tobytes(), E.to_dense().tobytes())
                 keypairlist.append(new_povm_effect_key_pair)
                 for measFid in available_meas_fid_list:
@@ -777,7 +775,6 @@ def create_prep_mxs(model, prep_fid_list, prep_cache=None):
                 except KeyError as err:
                     print('A (Rho, Circuit) pair is missing from the cache, all such pairs should be available is using the caching option.')
                     raise err                
-                    #outputMat[:, i] = _np.dot(model.sim.product(prepFid), rho.to_dense())
             outputMatList.append(outputMat)
 
     else:
@@ -836,13 +833,11 @@ def create_meas_mxs(model, meas_fid_list, meas_cache=None):
                 except KeyError as err:
                     print('A (POVM, Effect, Circuit) pair is missing from the cache, all such pairs should be available if using the caching option.')
                     raise err
-                    #outputMat[:, i] = _np.dot(E.to_dense(), model.sim.product(measFid))
             outputMatList.append(outputMat)
 
     else:
         for povm in model.povms.values():
             for E in povm.values():
-                #if isinstance(E, _ComplementPOVMEffect): continue  # complement is dependent on others
                 outputMat = _np.zeros([dimE, numFid], float)
                 for i, measFid in enumerate(meas_fid_list):
                     outputMat[:, i] = _np.dot(E.to_dense(), model.sim.product(measFid))
@@ -1774,10 +1769,6 @@ def _find_fiducials_greedy(model, fids_list, prep_or_meas, op_penalty=0.0,
             else: 
                 for fiducial in fids_list:
                     #calculate the score matrix
-                    #if prep_or_meas == 'prep':
-                    #    fidArrayList = create_prep_mxs(model, [fiducial], fid_cache)
-                    #elif prep_or_meas == 'meas':
-                    #    fidArrayList = create_meas_mxs(model, [fiducial], fid_cache)
                     current_score_mx= fiducial_compact_EVD_cache[fiducial]
                     current_score_gramian= fiducial_compact_EVD_cache[fiducial]@fiducial_compact_EVD_cache[fiducial].T
                     current_inv_trace = _np.trace(_np.linalg.pinv(current_score_gramian, hermitian=True))