TheEvergreenStateCollege / bioinformatics

Plant genome sequencing
2 stars 2 forks source link

Comment and strip out unneeded parts of suffix_tree.rs #45

Open AbyssalRemark opened 1 month ago

AbyssalRemark commented 1 month ago

While suffix_tree.rs is working. its clunky and has at least one unused component: empty_node_indicies: Vec<u32> which seems to be a mechanism for not deleting and allocating nodes over and over. Is unused. as fn delete_children(&mut self, node: usize) is unused. So there are a lot of extra checks for something that isn't in use (and also might complicate off loading to the database).

This is less then ideal. Sense the code is very lightly commented I think commenting this code will reveal what needs to be changed.