Issue224

Title Optimal LM heuristic segfaults on axioms
Priority bug Status resolved
Superseder Nosy List erez, florian, gabi, malte, silvia
Assigned To erez Keywords
Optional summary

Created on 2011-03-09.08:50:45 by silvia, last changed by florian.

Messages
msg3231 (view) Author: florian Date: 2014-07-24.12:18:17
The current revision in default correctly reports the error and exits with
EXIT_UNSUPPORTED (3).
msg3228 (view) Author: malte Date: 2014-07-24.10:36:23
Please test with the command-line Silvia mentions on a problem with axioms (e.g.
from psr-large?). If it exits with a clean error message, please close.
msg3227 (view) Author: florian Date: 2014-07-24.10:25:26
This issue seems to have a duplicate that is already fixed (issue380). Should we
close it or is there something else to it?
msg2509 (view) Author: gabi Date: 2013-06-25.14:58:29
see also issue247.
msg1266 (view) Author: erez Date: 2011-03-09.11:01:14
I see where this error is coming from - many references to g_operators, 
sometimes from functions which might return an axiom id 
(get_operators_including_eff) - but I think this should also be assigned to 
Emil.

Regarding checking about axioms/conditional effects/non-unit cost actions/zero 
cost actions/... - I suggest we implement some sort of capabilities system 
(similar to weka), which would give us a convenient way of checking all of 
these. I'll open a separate issue for this.
msg1265 (view) Author: silvia Date: 2011-03-09.09:27:05
I think I used

--search "astar(lmcount(lm_merged(lm_rhw(),lm_hm(m=1)),admissible=true),mpd=true)"

In this case it's the hm landmark generation that segfaults on axioms.
Of course it would be good if the hm landmarks would print an error message
rather than segfaulting. So maybe the title of this issue was not well chosen
and it should be "hm landmarks segfault on axioms".

But I think it would also be good though if there could be a check for axioms
(and a clean exit with error message) before any LM generation is even started.
Since the cost partitioning in the heuristic can't handle axioms, there's no
point in calculating landmarks at all, right? Is there a way that we can check
the capabilities of heuristics and other components like LM generations against
the requirements of the input, and print appropriate error messages right in the
beginning?

Emil is not on the tracker it seems. I can ask him if he'd like to join.
msg1264 (view) Author: erez Date: 2011-03-09.08:59:11
The first achievers are calculated in the LandmarksGraph - before the heuristic 
is created. What type of landmarks did you use here?
Can you attach the command line?
msg1262 (view) Author: silvia Date: 2011-03-09.08:50:44
While M&S and lmcut abort with a clean error message ("can't handle
axioms") right at the start, the optimal landmark count heuristic
doesn't do this. There is an error message saying that the
cost partitioning can't handle axioms, but this message comes too late
-- you only get to see it if you actually _get_ to the part of the
code that does the cost partitioning. If you use h_m landmarks, you
get a segmentation fault before getting that far. The seg fault happens when
the code tries to calculate first achievers.
History
Date User Action Args
2014-07-24 12:18:17floriansetstatus: chatting -> resolved
messages: + msg3231
2014-07-24 10:36:23maltesetmessages: + msg3228
2014-07-24 10:25:26floriansetnosy: + florian
messages: + msg3227
2013-06-25 14:58:29gabisetnosy: + gabi
messages: + msg2509
2011-03-09 11:01:14erezsetmessages: + msg1266
2011-03-09 09:27:05silviasetmessages: + msg1265
2011-03-09 08:59:11erezsetstatus: unread -> chatting
messages: + msg1264
2011-03-09 08:50:45silviacreate