Inconsistencies in the double factorized chemistry resource estimate costing function

[fdmalone]

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

https://github.com/quantumlib/OpenFermion/blob/a034397a13b1ea224e11746d4ee368cccf2bc962/src/openfermion/resource_estimates/df/compute_cost_df.py#L53

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?)