-
Notifications
You must be signed in to change notification settings - Fork 9
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
Fix Forte [Ambit Part] #53
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
This looks good to me. A comment on the style. Before this PR, load/save were implemented as free functions. I agree that save can be moved to a class function, but this now puts load/save in two different parts (free/member). Then perhaps it would a good idea to make save a static class function. I am curious how other codes handle this aspect. @jturney: any thoughts on this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Can I squash/merge?
That's fine with me, but I can also finish moving save/load to class functions if you want. Jet's policy has been that if tests are passing and he's approved, it's good to merge. |
Class functions++!
-Rob
…On Sat, Mar 26, 2022 at 1:00 PM Jonathon Misiewicz ***@***.***> wrote:
That's fine with me, but I can also finish moving save/load to class
functions if you want.
Jet's policy has been that if tests are passing and he's approved, it's
good to merge.
—
Reply to this email directly, view it on GitHub
<#53 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAULGP6I73TYTCUEYJOYBQTVB5UFBANCNFSM5RVXIQRA>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Then let's do class functions (good to see you here @robparrish!). |
Done. The static function |
This PR makes changes so that
Psi
can do DIIS on anambit.BlockedTensor
, as required by theforte
plugin.This PR accomplishes the following, as needed by this Psi PR:
load_blocked_tensor
method to Python asBlockedTensor.load
save
function for BlockedTensor to a method, and exposes it py-sideBlockedTensor
BlockedTensor
BlockedTensor
@fevangelista @lcyyork