-
Notifications
You must be signed in to change notification settings - Fork 21
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
BGL: Revise cost_recorder
event visitor
#82
Comments
marip8
changed the title
Revise
BGL: Revise Sep 7, 2021
cost_recorder
BGL event visitorcost_recorder
event visitor
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The
cost_recorder
event visitor makes a copy of the edge weight property map every time it is utilized in order to get the cost of an edge. It seems like there should be a more efficient way of getting the edge weight that avoids this unnecessary amount of copyingdescartes_light/descartes_light/solvers/include/descartes_light/solvers/bgl/impl/event_visitors.hpp
Line 63 in d56038b
Also, the class should probably be renamed to
distance_recorder
since it is actually recording the distance from the start vertex to each visited vertexThe text was updated successfully, but these errors were encountered: