Inconsistencies in the double factorized chemistry resource estimate costing function

[fdmalone]

the costing function computes br for outer prepare and inner prepare differently:

OpenFermion/src/openfermion/resource_estimates/df/compute_cost_df.py

Line 53 in a034397

for p in range(20):

Which would be fine but the br fed into cost two should be using the second (hardcoded value) of 7 to output into qrom for state preparation on the second register.

[mhucka]

(I'm going around and checking issue statuses.) @fdmalone could you let me know what the status of this is? (Should we unassignn you?)