Project

General

Profile

Actions

Pull request #846

open

Transistion-Based SST

Added by Jean-François Erdelyi over 8 years ago. Updated about 8 years ago.

Status:
New
Priority:
Normal
Assignee:
Jules Waldhart

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

Actions #1

Updated by Jean-François Erdelyi over 8 years ago

  • Assignee set to Jules Waldhart
Actions #2

Updated by Jules Waldhart about 8 years ago

need to remove one line to compile studio:
qtKinodynamic.cpp:18

Actions #3

Updated by Jules Waldhart about 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

Actions #4

Updated by Jules Waldhart about 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!

Actions #5

Updated by Jean-François Erdelyi about 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

Actions #6

Updated by Jules Waldhart about 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)...

Actions

Also available in: Atom PDF