-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Plan accepted while it should not #27
Comments
This one, on the other hand, is a significant problem. A long-lived bug has been uncovered! There is a (flawed) assumption deep in the machinery for tracking literals (and functions) that the concatenation of the strings that make up their names, together with the names of their grounded arguments, will uniquely identify them - not true if you have (P O AOB ) and (P OA OB ). This has now been corrected in my code base and I will push it to the repository as soon as I have access to it.
Cheers
Derek
…________________________________
From: mabdula <notifications@github.com>
Sent: 03 June 2018 17:06:36
To: KCL-Planning/VAL
Cc: Subscribed
Subject: [KCL-Planning/VAL] Plan accepted while it should not (#27)
Attached is a zip file containing a domain (domain_2.pddl) , an instance (inst_2.pddl) and an incorrect plan (plan_2.pddl). Running the "validate" executable on the three files gives the following output:
Checking plan: plan_2.pddl
Plan executed successfully - checking goal
Plan valid
Final value: 1
Successful plans:
Value: 1
plan_2.pddl 1
bug2.zip<https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FKCL-Planning%2FVAL%2Ffiles%2F2066109%2Fbug2.zip&data=01%7C01%7Cderek.long%40kcl.ac.uk%7Cdcc3b5d8e0fc4749ed3e08d5c96bfcbc%7C8370cf1416f34c16b83c724071654356%7C0&sdata=MDBplICzWoplSvYqAX2GMAxpOosth%2FELrdvOj0Az2vs%3D&reserved=0>
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub<https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FKCL-Planning%2FVAL%2Fissues%2F27&data=01%7C01%7Cderek.long%40kcl.ac.uk%7Cdcc3b5d8e0fc4749ed3e08d5c96bfcbc%7C8370cf1416f34c16b83c724071654356%7C0&sdata=evfW%2BEuwqEHm4swd03YRfSRHJc5Xd4cZPjz1hPva9ug%3D&reserved=0>, or mute the thread<https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAHSnVDwyO_x48BZf_CmodJZJfSLCG1ziks5t5AmMgaJpZM4UYIyq&data=01%7C01%7Cderek.long%40kcl.ac.uk%7Cdcc3b5d8e0fc4749ed3e08d5c96bfcbc%7C8370cf1416f34c16b83c724071654356%7C0&sdata=HHLN%2FBCBCzcUkzqdXbgBSLo96PEb0BQYPP7KxsTR1%2Bg%3D&reserved=0>.
|
This appears to be fixed in the latest codebase (about to be pushed to the repo). When the PR is accepted, this issue should be closed.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Attached is a zip file containing a domain (domain_2.pddl) , an instance (inst_2.pddl) and an incorrect plan (plan_2.pddl). Running the "validate" executable on the three files gives the following output:
Checking plan: plan_2.pddl
Plan executed successfully - checking goal
Plan valid
Final value: 1
Successful plans:
Value: 1
plan_2.pddl 1
bug2.zip
Cheers,
Mohammad
The text was updated successfully, but these errors were encountered: