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
Issue1070 #141
Issue1070 #141
Conversation
e2c2ae3
to
bc33b9b
Compare
d4bb4a6
to
b9f398d
Compare
return false. If a simple landmark can be achieved, return only operators | ||
that achieve simple landmarks, else return operators that achieve | ||
disjunctive landmarks */ | ||
bool LandmarkCountHeuristic::generate_helpful_actions(const State &state) { |
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.
We should clean up the signature: there is no longer a reason to have this be a non-void function. The general preference is for mutating functions like these ones to be void; the return value is not obvious from the name and is unused. Also, the function name should be generate_preferred_operators; "helpful actions" is an EHC/FF term. If we make this void, the following comment also should be adjusted.
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.
This also entails changing the variable names that use "ha", and I wouldn't mix "action" and "operator" in the same comment, as in "Find actions that... return operators that...". Also, the function doesn't actually return operators.
060f268
to
514535d
Compare
…t of reached landmarks.
514535d
to
079d9d4
Compare
No description provided.