From e6d0afd1e2cfa1b0d0d443af8c6448bbdf08f5c0 Mon Sep 17 00:00:00 2001 From: Kuan Butts Date: Mon, 5 Mar 2018 21:52:55 -0800 Subject: [PATCH] dont need the renaming of the sid col in the first place? --- peartree/graph.py | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/peartree/graph.py b/peartree/graph.py index a362933..5a43268 100644 --- a/peartree/graph.py +++ b/peartree/graph.py @@ -230,12 +230,6 @@ def make_synthetic_system_network( for key, val in sid_lookup_sub.items(): sid_lookup[key] = val - # TODO(kuanb): Can this be consolidated with _add_nodes_and_edges? - # Update node stop_id name to prepare it for generate_cross_feed_edges - # check steps, which require name to include graph name as well - nodes['orig_stop_id'] = nodes['stop_id'].copy() - nodes['stop_id'] = '{}_'.format(name) + nodes['stop_id'] - # Then add to the running tally of nodes if all_nodes is None: all_nodes = nodes.copy() @@ -247,11 +241,6 @@ def make_synthetic_system_network( cross_feed_edges = generate_cross_feed_edges(G, all_nodes, exempt_nodes, connection_threshold) - - # TODO(kuanb): This is a hack to deal with the inconsistent use of the - # sid_lookup table - should see if we can remove its need - # at this stage and just have full stop_id already here - nodes['stop_id'] = nodes['orig_stop_id'].copy() # Mutates the G network object _add_cross_feed_edges(G, sid_lookup, cross_feed_edges, walk_speed_kmph)