-
Notifications
You must be signed in to change notification settings - Fork 121
Adding optimizer: ExcitationSolve #433
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: devel
Are you sure you want to change the base?
Conversation
|
errors in test seem to be unrelated to tq, but an internal issue in excitation_solve |
|
A few errors are similar to this: /opt/hostedtoolcache/Python/3.9.24/x64/lib/python3.9/site-packages/excitationsolve/excitation_solve.py:7: in <module>
parameter_variations: list | np.ndarray,
E TypeError: unsupported operand type(s) for |: 'type' and 'type'excitationsolve uses the Is it possible to use python3.10 or newer in the git runner or should I make excitationsolve compatible with python3.9? |
|
Ah, thanks for digging that up. |
Update recommended Python version in installation instructions.
Comment out h5py installation to prevent conflicts.
|
Issue is on the openfermion end. Issues with numpy.int64 which is not recognized as int anymore. Will remove affected transformations for now. |
Comment out problematic test cases related to openfermion.
too much work to maintain
Added a check for zero norm before eigenvalue computation.
Added debug print statements for troubleshooting.
Hey everyone,
I'd like to add the ExcitationSolve (code, preprint) optimizer to tequila. See a code example below using the implemented ExcitationSolve optimizer.
Open questions:
historymember variable correctly.Code example