Issue451

Title Solvable problem marked as unsolvable
Priority bug Status resolved
Superseder Nosy List gabi, malte, mystique
Assigned To malte Keywords
Optional summary

Created on 2014-08-11.10:38:49 by mystique, last changed by mystique.

Files
File name Uploaded Type Edit Remove
problem.zip mystique, 2014-08-11.10:38:49 application/zip
Messages
msg3329 (view) Author: mystique Date: 2014-08-19.16:39:32
Seems to work fine now! Thanks! :)
msg3325 (view) Author: gabi Date: 2014-08-19.11:24:49
Since Malte agreed, I merged this into the public default branch.

The patch does not affect the translator output on the standard benchmark tasks
and works for the given test instance but it is in a part of the code that we do
not touch very often. If you still encounter problems on other test instances,
feel free to reopen the issue.
msg3324 (view) Author: mystique Date: 2014-08-19.10:43:25
Is there a fix I can test? I found few other similar problems (some even without
either) where I can test it...
msg3313 (view) Author: malte Date: 2014-08-12.20:44:23
I don't really know the old code well enough, so if you're happy with the change
and it passes all the tests, I'm happy to see it merged without further review.
msg3311 (view) Author: gabi Date: 2014-08-12.16:12:58
The fix is ready for reviewing. It does not have significant impact on the
translation time and the output on the existing benchmarks tasks is unchanged
(except for a few trucks-strips instances where the differences are presumably
due to the timeout of the invariant synthesis triggering at different states).
msg3302 (view) Author: malte Date: 2014-08-11.15:20:43
As Gabi wrote, the "either" isn't the main issue here, but I should point out
that it is not supported at all. We're curently missing the error reporting to
clearly indicate all cases where the PDDL input is not supported, unfortunately.
If you want to track the progress on that, it's issue323.
msg3301 (view) Author: gabi Date: 2014-08-11.11:05:42
The attached example is wonderfully simple. Thanks a lot!

The invariant finder has some issue with constants. If I include the elevator as
an additional action parameter, everything works.
msg3300 (view) Author: gabi Date: 2014-08-11.10:53:40
This looks like a problem in the invariant synthesis. The translator finds a
mutex group
[Atom at(?X, n8)]
which implies that there can only be one passenger at floor n8.
I will have a look into this.

Apart from this, Fast Downward does not fully support (either ...) types (cf.
http://www.fast-downward.org/PddlSupport). But this appears not to be the source
of the problem.
msg3299 (view) Author: mystique Date: 2014-08-11.10:38:49
Hi,
for the attached problem, no solution has been found even though it has a solution.

With any of these small changes of the problem, the solution is found:
 - delete one of goals
 - delete unnecessary fact (at p3 n8) from init
 - change second goal (at p6 n8) -> (at p6 n5) 

It seems that there is something wrong in translator, since the generated SAS is
empty.

Thanks for help!
History
Date User Action Args
2014-08-19 16:39:32mystiquesetmessages: + msg3329
2014-08-19 11:24:49gabisetstatus: reviewing -> resolved
messages: + msg3325
2014-08-19 10:43:25mystiquesetmessages: + msg3324
2014-08-12 20:44:23maltesetmessages: + msg3313
2014-08-12 16:12:58gabisetstatus: chatting -> reviewing
assignedto: gabi -> malte
messages: + msg3311
2014-08-11 15:20:43maltesetnosy: + malte
messages: + msg3302
2014-08-11 11:05:42gabisetmessages: + msg3301
2014-08-11 10:53:40gabisetstatus: unread -> chatting
assignedto: gabi
messages: + msg3300
nosy: + gabi
2014-08-11 10:38:49mystiquecreate