add better and more precise comments to Restrictionmap

This commit is contained in:
Dennis Luxen 2014-05-22 14:53:53 +02:00
parent 0c66f84555
commit 7d7cce5c72
2 changed files with 27 additions and 39 deletions

View File

@ -39,8 +39,8 @@ RestrictionMap::RestrictionMap(const std::shared_ptr<NodeBasedDynamicGraph> &gra
const std::vector<TurnRestriction> &input_restrictions_list) const std::vector<TurnRestriction> &input_restrictions_list)
: m_count(0), m_graph(graph) : m_count(0), m_graph(graph)
{ {
// decompose restirction consisting of a start, via and end note into a start-edge // decompose restriction consisting of a start, via and end node into a
// and all end-nodes // a pair of starting edge and a list of all end nodes
for (auto &restriction : input_restrictions_list) for (auto &restriction : input_restrictions_list)
{ {
m_restriction_start_nodes.insert(restriction.fromNode); m_restriction_start_nodes.insert(restriction.fromNode);
@ -77,27 +77,22 @@ RestrictionMap::RestrictionMap(const std::shared_ptr<NodeBasedDynamicGraph> &gra
} }
} }
/** // Replace end v with w in each turn restriction containing u as via node
* Replace end v with w in each turn restriction containing u as via node
*
* Note: We need access to node based graph.
*/
void RestrictionMap::FixupArrivingTurnRestriction(const NodeID u, const NodeID v, const NodeID w) void RestrictionMap::FixupArrivingTurnRestriction(const NodeID u, const NodeID v, const NodeID w)
{ {
BOOST_ASSERT(u != std::numeric_limits<unsigned>::max()); BOOST_ASSERT(u != SPECIAL_NODEID);
BOOST_ASSERT(v != std::numeric_limits<unsigned>::max()); BOOST_ASSERT(v != SPECIAL_NODEID);
BOOST_ASSERT(w != std::numeric_limits<unsigned>::max()); BOOST_ASSERT(w != SPECIAL_NODEID);
if (!RestrictionStartsAtNode(u)) if (!RestrictionStartsAtNode(u))
{ {
return; return;
} }
// find all possible start edges // find all potential start edges
// it is more efficent to get a (small) list of potential start edges // it is more efficent to get a (small) list of potential start edges than iterating over all buckets
// than iterating over all buckets
std::vector<NodeID> predecessors; std::vector<NodeID> predecessors;
for (EdgeID current_edge_id : m_graph->GetAdjacentEdgeRange(u)) for (const EdgeID current_edge_id : m_graph->GetAdjacentEdgeRange(u))
{ {
const EdgeData &edge_data = m_graph->GetEdgeData(current_edge_id); const EdgeData &edge_data = m_graph->GetEdgeData(current_edge_id);
const NodeID target = m_graph->GetTarget(current_edge_id); const NodeID target = m_graph->GetTarget(current_edge_id);
@ -127,21 +122,19 @@ void RestrictionMap::FixupArrivingTurnRestriction(const NodeID u, const NodeID v
} }
} }
/** // Replaces start edge (v, w) with (u, w). Only start node changes.
* Replaces the start edge (v, w) with (u, w), only start node changes.
*/
void RestrictionMap::FixupStartingTurnRestriction(const NodeID u, const NodeID v, const NodeID w) void RestrictionMap::FixupStartingTurnRestriction(const NodeID u, const NodeID v, const NodeID w)
{ {
BOOST_ASSERT(u != std::numeric_limits<unsigned>::max()); BOOST_ASSERT(u != SPECIAL_NODEID);
BOOST_ASSERT(v != std::numeric_limits<unsigned>::max()); BOOST_ASSERT(v != SPECIAL_NODEID);
BOOST_ASSERT(w != std::numeric_limits<unsigned>::max()); BOOST_ASSERT(w != SPECIAL_NODEID);
if (!RestrictionStartsAtNode(u)) if (!RestrictionStartsAtNode(u))
{ {
return; return;
} }
auto restriction_iterator = m_restriction_map.find({v, w}); const auto restriction_iterator = m_restriction_map.find({v, w});
if (restriction_iterator != m_restriction_map.end()) if (restriction_iterator != m_restriction_map.end())
{ {
const unsigned index = restriction_iterator->second; const unsigned index = restriction_iterator->second;
@ -154,18 +147,16 @@ void RestrictionMap::FixupStartingTurnRestriction(const NodeID u, const NodeID v
} }
} }
/* // Check if edge (u, v) is the start of any turn restriction.
* Check if the edge (u, v) is contained in any turn restriction. // If so returns id of first target node.
* If so returns id of first target node.
*/
NodeID RestrictionMap::CheckForEmanatingIsOnlyTurn(const NodeID u, const NodeID v) const NodeID RestrictionMap::CheckForEmanatingIsOnlyTurn(const NodeID u, const NodeID v) const
{ {
BOOST_ASSERT(u != std::numeric_limits<unsigned>::max()); BOOST_ASSERT(u != SPECIAL_NODEID);
BOOST_ASSERT(v != std::numeric_limits<unsigned>::max()); BOOST_ASSERT(v != SPECIAL_NODEID);
if (!RestrictionStartsAtNode(u)) if (!RestrictionStartsAtNode(u))
{ {
return std::numeric_limits<unsigned>::max(); return SPECIAL_NODEID;
} }
auto restriction_iter = m_restriction_map.find({u, v}); auto restriction_iter = m_restriction_map.find({u, v});
@ -181,18 +172,15 @@ NodeID RestrictionMap::CheckForEmanatingIsOnlyTurn(const NodeID u, const NodeID
} }
} }
} }
return std::numeric_limits<unsigned>::max(); return SPECIAL_NODEID;
} }
/** // Checks if turn <u,v,w> is actually a turn restriction.
* Checks if the turn described by start u, via v and targed w is covert by any turn restriction.
*/
bool RestrictionMap::CheckIfTurnIsRestricted(const NodeID u, const NodeID v, const NodeID w) const bool RestrictionMap::CheckIfTurnIsRestricted(const NodeID u, const NodeID v, const NodeID w) const
{ {
// SimpleLogger().Write(logDEBUG) << "checking turn <" << u << "," << v << "," << w << ">"; BOOST_ASSERT(u != SPECIAL_NODEID);
BOOST_ASSERT(u != std::numeric_limits<unsigned>::max()); BOOST_ASSERT(v != SPECIAL_NODEID);
BOOST_ASSERT(v != std::numeric_limits<unsigned>::max()); BOOST_ASSERT(w != SPECIAL_NODEID);
BOOST_ASSERT(w != std::numeric_limits<unsigned>::max());
if (!RestrictionStartsAtNode(u)) if (!RestrictionStartsAtNode(u))
{ {
@ -203,7 +191,7 @@ bool RestrictionMap::CheckIfTurnIsRestricted(const NodeID u, const NodeID v, con
if (restriction_iter != m_restriction_map.end()) if (restriction_iter != m_restriction_map.end())
{ {
const unsigned index = restriction_iter->second; const unsigned index = restriction_iter->second;
auto &bucket = m_restriction_bucket_list.at(index); const auto &bucket = m_restriction_bucket_list.at(index);
for (const RestrictionTarget &restriction_target : bucket) for (const RestrictionTarget &restriction_target : bucket)
{ {
if ((w == restriction_target.first) && // target found if ((w == restriction_target.first) && // target found
@ -216,6 +204,7 @@ bool RestrictionMap::CheckIfTurnIsRestricted(const NodeID u, const NodeID v, con
return false; return false;
} }
// check of node is the start of any restriction
bool RestrictionMap::RestrictionStartsAtNode(const NodeID node) const bool RestrictionMap::RestrictionStartsAtNode(const NodeID node) const
{ {
if (m_restriction_start_nodes.find(node) == m_restriction_start_nodes.end()) if (m_restriction_start_nodes.find(node) == m_restriction_start_nodes.end())

View File

@ -39,8 +39,7 @@ SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
#include <unordered_map> #include <unordered_map>
#include <unordered_set> #include <unordered_set>
// Efficent look up if an edge is the start + via node of a TurnRestriction
// Make it efficent to look up if an edge is the start + via node of a TurnRestriction
// EdgeBasedEdgeFactory decides by it if edges are inserted or geometry is compressed // EdgeBasedEdgeFactory decides by it if edges are inserted or geometry is compressed
class RestrictionMap class RestrictionMap
{ {