Title landmark code refactoring miscellanea
Priority wish Status resolved
Superseder Nosy List erez, malte, silvan, silvia
Assigned To malte Keywords
Optional summary

Created on 2011-04-05.00:50:38 by malte, last changed by malte.

msg3461 (view) Author: malte Date: 2014-09-19.17:46:35
Forgot to close this one.
msg1493 (view) Author: malte Date: 2011-08-07.20:40:14
msg1311 (view) Author: erez Date: 2011-04-05.09:24:04
In that case, I will not do anything about deleting the RPG Search LM Graph 
msg1309 (view) Author: malte Date: 2011-04-05.00:50:38
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:


 * 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.
Date User Action Args
2014-09-19 17:46:35maltesetstatus: chatting -> resolved
messages: + msg3461
2011-08-07 20:40:14maltesetassignedto: silvan -> malte
messages: + msg1493
2011-04-05 09:24:04erezsetnosy: + erez
messages: + msg1311
2011-04-05 01:00:53silviasetnosy: + silvia
2011-04-05 00:50:38maltecreate