OpenFermion-FQE icon indicating copy to clipboard operation
OpenFermion-FQE copied to clipboard

Consider a unique definition for Wavefunction

Open rmlarose opened this issue 5 years ago • 2 comments

Wavefunction is currently multiply defined, once as the class and once as a getter function for the class. This caused a few issues with mypy, and while they are easily resolved by importing the correct Wavefunction, there is a chance for confusion. Perhaps refactor the function to get_wavefunction or a similar name?

rmlarose avatar Jan 05 '21 21:01 rmlarose

This causes additional problems with Pylance in VSCode. It really would be a nice change even if it's annoying to break people's code.

wjhuggins avatar Jun 22 '22 23:06 wjhuggins

Would a PR that does this be approved?

wjhuggins avatar Jun 22 '22 23:06 wjhuggins