Added normal ordering to, simplified multiplication in FermionOperator

eclipse / xacc

XACC - eXtreme-scale Accelerator programming framework

https://xacc.readthedocs.io

BSD 3-Clause "New" or "Revised" License

163 stars 84 forks source link

Closed danclaudino closed 1 year ago

danclaudino commented 1 year ago

Added FermionOperator::normalOrder() to normal order FermionOperators (not in-place)
Simplified FermionTerm::operator*=() to bypass redundancies (further simplification can be done by normal ordering the resulting FermionOperator or by calling ObservableTransform::transform())
Plugin chemistry observables cast to FermionOperator within getObservable(). This is done in python and now in C++.
Added/adjusted tests

Signed-off-by: Daniel Claudino 6d3@ornl.gov

1tnguyen commented 1 year ago

Thanks, @danclaudino.

The CI failures are unrelated (due to some updates in the GitHub CI environment, fixed in https://github.com/eclipse/xacc/pull/567)