-
Notifications
You must be signed in to change notification settings - Fork 137
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
Move NonLocalTOperator objects from Hamiltonian to Drivers #5137
Conversation
4d87314
to
f8dadb6
Compare
@@ -112,6 +111,16 @@ class QMCDriverNew : public QMCDriverInterface, public MPIObjectBase | |||
//xmlNodePtr walker_logs_xml; | |||
|
|||
protected: | |||
class ContextForSteps | |||
{ |
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.
Can you add a description?
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.
will do.
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.
fixed.
{} | ||
|
||
///non local operator | ||
NonLocalTOperator non_local_ops; |
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.
Can this be protected or private?
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.
ContextForSteps and its derived class are intended to be just a struct. Marking it private and adding an access function are unnecessary.
6cfff41
to
96715a4
Compare
Test this please |
Review after #5135
Proposed changes
What type(s) of changes does this code introduce?
Does this introduce a breaking change?
What systems has this change been tested on?
laptop
Checklist