Add in combinatorial code #68
Conversation
dkhofer
force-pushed
the
combinatorial-designs
branch
from
59f83a9
to
d03e30f
Compare
| @@ -483,6 +467,30 @@ impl Edge { | |||
|
|
|||
| (graph, edges_by_node_pair) | |||
| } | |||
|
|
|||
| pub fn boundary_edges_from_sequences( | |||
There was a problem hiding this comment.
I didn't like returning two separate things from the above method, felt like splitting out the boundary edge computation into a separate method
|
|
||
| let mut parts_reader = fasta::io::reader::Builder.build_from_path(parts_file_path)?; | ||
|
|
||
| let path = Path::get_paths( |
There was a problem hiding this comment.
Currently assumes there is only one path with the input name in the db, need to think about how to better handle this. I don't want users to have to specify a block group ID
There was a problem hiding this comment.
could do a blockgroup name + path instead of id. BG would be something like "X-vector", path would be backbone or something
| new_edges.insert(edge); | ||
| } | ||
|
|
||
| for part1 in &parts1 { |
There was a problem hiding this comment.
Creates all the edges for the combinatorial design unconditionally. Can add in a flag to just add the specified part orderings if Bob wants
| BlockGroupEdge::bulk_create(conn, path.block_group_id, &new_edge_ids); | ||
|
|
||
| let path_changes_count = parts1.len() * parts2.len(); | ||
| let summary_str = format!("{path_name}: {path_changes_count} changes.\n"); |
There was a problem hiding this comment.
Not sure exactly what to put here, I'm open to suggestions
| path_name: Option<String>, | ||
| /// The start coordinate for the region to add the library to | ||
| #[arg(short, long)] | ||
| start: Option<i64>, |
There was a problem hiding this comment.
VCF imports fail now on a collision between -s for sample and -s for start
thread 'main' panicked at /Users/bvh/.cargo/registry/src/index.crates.io-6f17d22bba15001f/clap_builder-4.5.17/src/builder/debug_asserts.rs:112:17: Command update: Short option names must be unique for each argument, but '-s' is in use by both 'sample' and 'start'
No description provided.