Change "Myplanner::connect_to_graph" about hqmp HOT 6 CLOSED

Of course assertions only compile on debug.
Let's define our own assertion that compiles according to a flag. We can
make it pause execution with a message (i.e. print something and then call
getchar) to get our attention.
Obviously we'll turn it off before submitting.

2012/8/27 noaminbar510 [email protected]

Having the following assumptions:

1. Source, targets, and all aditional targets are sampled with both
   connectors and layers.
2. Every motion_step is inside fsc's and therfore every point in a motion
   step belongs to a node(or an edge?) in the graph.

Thus, connect to graph should do nothing, and in myplanner::query",
perturbed_source == source, perturbed_target==target.
Should add an assertion/log message when connect_to_graph does return
differnt points.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/25.

from hqmp.

I looked on "assert.h" code and felt a bit sad, show me this tommorow first thing on the morning, as there are more pending task needed to be asserted.

from hqmp.

Implemented in f79fdf6

2012/8/28 noaminbar510 [email protected]

I looked on "assert.h" code and felt a bit sad, show me this tommorow
first thing on the morning, as there are more pending task needed to be
asserted.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/25#issuecomment-8070971.

from hqmp.

Can we close this issue?

from hqmp.

Ping.

from hqmp.

removed incorrect assertion on commit 0313a24.
closing the issue.

from hqmp.