Merge pull request #2383 from jeizenga/mod-with-paths

Sync paths correctly in VG::expand_context
vgteam · Aug 10, 2019 · 41901bc · 41901bc · adamnovak · Aug 10, 2019
2 parents 4302dff + b77a012
commit 41901bc
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/path.cpp b/src/path.cpp
@@ -269,6 +269,7 @@ bool Paths::has_mapping(const string& name, int32_t rank) {
 }
 
 void Paths::append_mapping(const string& name, const mapping_t& m, bool warn_on_duplicates) {
+
     // get or create the path with this name
     list<mapping_t>& pt = get_create_path(name);
     // now if we haven't already supplied a mapping

diff --git a/src/vg.cpp b/src/vg.cpp
@@ -1968,7 +1968,11 @@ void VG::expand_context_by_steps(VG& g, size_t steps, bool add_paths) {
                     }
                 }
             });
-        g.sync_paths();
+        g.paths.sort_by_mapping_rank();
+        g.paths.rebuild_mapping_aux();
+
+        // store paths in graph
+        g.paths.to_graph(g.graph);
     }
 }