Some suggested changes to the landmark code came up in the discussion of
issue222. We didn't want to make them in the default code since this would very
likely make merging the changes in Silvan's landmark refactoring branch harder,
so I'm collecting them here to apply later:
Exploration::plan_for_disj
* Should return a bool that states whether the problem is relaxed
solvable or not, rather than the current int. I'd suggest "true" for
solvable. Its only caller must then be adapted to take this change
into account.
* Needs a comment that states that the main purpose of the method is to
mark preferred operators, via the "exported_ops" attribute.
Exploration::compute_ff_heuristic_with_excludes should be deleted.
It is only used by LandmarksGraph::relaxed_plan_length_without, which should
also be deleted.
This in turn is only used by the "RPG search" LM graph factory, which should
also be deleted.
|