Pull request #846
openTransistion-Based SST
Description
Add T-SST and new metric for kinodynamic planning
/home/jferdely/Public/pullreq/planners.git branch : T-SST_pullreq
/home/jferdely/Public/pullreq/studio.git branch : Kino-Cost_pullreq
/home/jferdely/Public/pullreq/lib.git branch : Kino-Cost_pullreq
Updated by Jean-François Erdelyi over 8 years ago
- Assignee set to Jules Waldhart
Updated by Jules Waldhart over 8 years ago
need to remove one line to compile studio:
qtKinodynamic.cpp:18
Updated by Jules Waldhart over 8 years ago
usage of loggers:
Anything intended only for logging must be called inside the [LOG]M3D_* macros.
e.g. do:
M3D_DEBUG("message "<<function_call(x)<<"\n"<<...);
and not something like:
ss<<"message"; ss<<function_call(x)<<endl;... M3D_DEBUG(ss.str());
Happens in TransitionSST.cpp
Updated by Jules Waldhart over 8 years ago
Several little things bother me.
- TSSTALGORITHM and SSTALGORITHM #defines => use planEnv flags
- W1 and W2 #defines in TransitionSST.cpp => must be parameters, and check that a weight is not nul before computing the associated part.
Looks ok otherwise!
Updated by Jean-François Erdelyi over 8 years ago
Up to date
/home/jferdely/Public/pullreq/planners.git branch : T-SST_pullreq
/home/jferdely/Public/pullreq/studio.git branch : T-SST_pullreq
/home/jferdely/Public/pullreq/lib.git branch : Kino-Cost_pullreq
Updated by Jules Waldhart over 8 years ago
Apparently you are messing up with the node neighbors. They are supposed to be a list of nodes connected to that node by edge, but you manually add neighbors (in SST) with Node::addNeighbor(Node*) when that is not the case. You need to fix this quickly to get graph class consistent again (I need it for the changes I'm trying to do)...