-
Notifications
You must be signed in to change notification settings - Fork 158
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
WIP: Remove harcoded path in TFDPlannerInterface #209
base: master
Are you sure you want to change the base?
Conversation
Looks sensible - just going to test on my machine. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The planner interface is not working very well for me - using a straightforward planner command:
"planners/tfd-src-0.4/downward/plan DOMAIN PROBLEM solution.pddl"
I get a few errors instead of a plan:
"./plan.py: No such file or directory"
The main reason being that the working directory of the planner interface is different from the TFD folder. In order to make this planner straightforward to use I would have to add something to the planner command.
Is there a way that the planner interface can be updated to automatically recognise the path of TFD and to handle this for the user?
Hello, I will take a look into it. What I did as a workaround was to modify tfd plan.py to remove the relative paths, here is how it looks like:
This solution was shared in google groups, I will take a look and see if this can be done in a nicer way and without modifying the original script. |
I made some changes and need some intermediate feedback, this is to check if I am going in the right direction. If so, I can adapt the other planner interfaces to this strategy. With this changes the tfd planner command would look like this:
|
This change is needed due to the removal of rosplan_demos and a workaround that the original author had set in the file.