-
Notifications
You must be signed in to change notification settings - Fork 89
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
[jsk_2013_jsk_2013_04_pr2_610] demo does not work with downward planner #1286
Comments
Examples of pddl_planner works well in my environments. |
This was referenced Apr 2, 2020
@sktometometo do you find why it failing. I think some text file? return value of service? has wrong syntax. can you debug this?
|
k-okada
added a commit
to k-okada/jsk_planning
that referenced
this issue
Mar 3, 2021
jsk-ros-pkg/jsk_planning#88 should fix this issue |
k-okada
added a commit
to jsk-ros-pkg/jsk_planning
that referenced
this issue
Oct 21, 2022
add test to reproduce jsk-ros-pkg/jsk_demos#1286
k-okada
added a commit
to k-okada/jsk_planning
that referenced
this issue
Dec 4, 2023
jsk-ros-pkg#88 changes 'sp.Popen(command' to 'sp.Popen(" ".join(command)', this assumes `planner_option` uses `--search &quat;iterated([lazy_greedy([hff,hlm], preferred=[hff,hlm]), ...)&quat;` but some launch file uses `--search iterated([lazy_greedy([hff,hlm],preferred=[hff,hlm]), ...)`, without &quat; and spaces, and jsk_planning 0.1.13 did not work this such eample(https://github.com/jsk-ros-pkg/jsk_demos/blob/ab0360b5580e77ca70006ce505497894fe4ac0d2/jsk_2013_04_pr2_610/test/test-demo-plan.test#L10), jsk-ros-pkg/jsk_demos#1286 this fix uses shlex.split() to keep quated substrings and uses Popen(command, stead of Popen(" ".join(command), to input quated argument as one word.
k-okada
added a commit
to k-okada/jsk_planning
that referenced
this issue
Dec 5, 2023
but some launch file uses `--search iterated([lazy_greedy([hff,hlm],preferred=[hff,hlm]), ...)`, such as jsk-ros-pkg/jsk_demos#1286 this test check both cases
k-okada
added a commit
to k-okada/jsk_planning
that referenced
this issue
Dec 5, 2023
but some launch file uses `--search iterated([lazy_greedy([hff,hlm],preferred=[hff,hlm]), ...)`, such as jsk-ros-pkg/jsk_demos#1286 this test check both cases
k-okada
added a commit
to k-okada/jsk_planning
that referenced
this issue
Dec 5, 2023
jsk-ros-pkg#88 changes 'sp.Popen(command' to 'sp.Popen(" ".join(command)', this assumes `planner_option` uses `--search &quat;iterated([lazy_greedy([hff,hlm], preferred=[hff,hlm]), ...)&quat;` but some launch file uses `--search iterated([lazy_greedy([hff,hlm],preferred=[hff,hlm]), ...)`, without &quat; and spaces, and jsk_planning 0.1.13 did not work this such eample(https://github.com/jsk-ros-pkg/jsk_demos/blob/ab0360b5580e77ca70006ce505497894fe4ac0d2/jsk_2013_04_pr2_610/test/test-demo-plan.test#L10), jsk-ros-pkg/jsk_demos#1286 this fix uses shlex.split() to keep quated substrings and uses Popen(command, stead of Popen(" ".join(command), to input quated argument as one word.
@sktometometo I encountered same problem and fixed. Please use latest jsk_planning (or deb versin of 0.1.12 for melodic) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This demo does not work with downward planner.
case 1 ( demo_tray.launch + downward planner )
case 2 ( demo_tray.launch + ffha planner )
And the demo works well.
The failure of case 1 seems to be because of some kind of syntax error.
The text was updated successfully, but these errors were encountered: