Issue90

Title Solvable problem reported unsolvable by translator
Priority bug Status resolved
Superseder Nosy List gabi, malte, patrik
Assigned To gabi Keywords 1.0
Optional summary

Created on 2010-06-16.02:40:35 by patrik, last changed by gabi.

Files
File name Uploaded Type Edit Remove
test1-domain.pddl patrik, 2010-06-16.02:41:21 application/octet-stream
test1-problem.pddl patrik, 2010-06-16.02:41:38 application/octet-stream
test1-problem.soln.2 patrik, 2010-06-16.02:42:18 application/octet-stream
test1-problem.soln.3 patrik, 2010-06-16.02:42:39 application/octet-stream
tg-test1-domain.pddl patrik, 2010-06-16.02:40:35 application/octet-stream
tg-test1-problem.pddl patrik, 2010-06-16.02:41:04 application/octet-stream
Messages
msg1057 (view) Author: gabi Date: 2011-01-05.19:39:11
Fixed.
msg1053 (view) Author: malte Date: 2011-01-05.19:25:06
This should be fixed as part of the issue132 branch. (The issue is not related
to issue132, but the fix was included in that branch by accident.)

Setting this to "testing". Gabi, if you can verify that this now works properly,
feel free to set it to resolved.
msg858 (view) Author: malte Date: 2010-12-14.17:02:20
Universally quantified effects are a very rarely used feature in our benchmark
set, and nontrivial type hierarchies are not that all that common either.

I think this one place (the only place, I think?) where our grounding
algorithm's implementation differs a bit from how it is described in the paper.
In the long run, it may be a good idea to change it to the way it's described in
the paper, which is conceptually cleaner and should be no less efficient. Can
you add a TODO to the code?
msg857 (view) Author: gabi Date: 2010-12-14.16:54:23
Yes, the universally quantified effects are the problem:
The instantiate-methods get a type-to-objects dictionary which is
used to identify the possible instantiations of the quantified variables.

This dictionary did not contain entries for supertypes, yet.
This is actually very easy to fix (like 3 lines), but I am surprised
that nobody encountered this bug before.

BTW: Thank you Patrik for your really helpful bug reports.
With such small examples it is almost a pleasure to fix the bugs.
msg856 (view) Author: malte Date: 2010-12-14.16:46:13
Wonderful, thanks! I'm surprised about the supertypes issue. Does this have
something to do with universally quantified effects? I saw some unexpected
weirdnesses in the instantiation code there yesterday.
msg855 (view) Author: gabi Date: 2010-12-14.16:42:27
Case #1 is the result of two bugs:

First, the invariant synthesis is not correct (already solved in issue132 but
not yet in the trunk).

Second, supertypes of types were not considered when instantiating the actions.
This lead to action (FaultSensor-sense_fault-__SPONTANEOUS__-fault FS1) being
removed.

Solving these bugs also resolves case #2.

Malte, as soon as the solution of issue132 is in the trunk, I
can branch from that version and fix the second bug.
msg339 (view) Author: patrik Date: 2010-06-16.03:02:24
Update wrt case #2: VAL (version 4.2.04) fails to validate the plan with cost 2
(soln.2). I'm pretty sure this is a VAL bug though.
msg338 (view) Author: patrik Date: 2010-06-16.02:40:35
There are potentially two separate issues here.

Case #1: tg-test1 (-domain/-problem) is solvable (solutions found by at least
three other planners) but the fd translator reports it as unsolvable (goal
simplified to false).

Case #2: test1 (-domain/-problem). This problem has a solution with cost 2 (I
believe that to be optimal), but fd search with options "bo" (i.e., blind
optimal search) finds a solution with cost 10.
History
Date User Action Args
2011-01-05 19:39:11gabisetstatus: testing -> resolved
messages: + msg1057
2011-01-05 19:25:06maltesetstatus: in-progress -> testing
messages: + msg1053
2010-12-14 17:02:20maltesetmessages: + msg858
2010-12-14 16:54:23gabisetmessages: + msg857
2010-12-14 16:46:13maltesetmessages: + msg856
2010-12-14 16:42:33gabisetstatus: chatting -> in-progress
messages: + msg855
2010-10-15 19:04:29maltesetassignedto: malte -> gabi
nosy: + gabi
2010-06-18 00:14:13maltesetkeyword: + 1.0
2010-06-16 14:08:20maltesetnosy: + malte
assignedto: malte
2010-06-16 03:02:24patriksetstatus: unread -> chatting
messages: + msg339
2010-06-16 02:42:39patriksetfiles: + test1-problem.soln.3
2010-06-16 02:42:18patriksetfiles: + test1-problem.soln.2
2010-06-16 02:41:38patriksetfiles: + test1-problem.pddl
2010-06-16 02:41:21patriksetfiles: + test1-domain.pddl
2010-06-16 02:41:04patriksetfiles: + tg-test1-problem.pddl
2010-06-16 02:40:35patrikcreate