fix: Avoid re-calculating md5sum on clone and conversion to KmerMinHa…

…shBTree
sourmash-bio · Nov 11, 2024 · d488294 · d488294
1 parent e86c8a8
commit d488294
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/src/core/src/sketch/minhash.rs b/src/core/src/sketch/minhash.rs
@@ -1006,7 +1006,7 @@ impl Clone for KmerMinHashBTree {
             mins: self.mins.clone(),
             abunds: self.abunds.clone(),
             current_max: self.current_max,
-            md5sum: Mutex::new(Some(self.md5sum())),
+            md5sum: Mutex::new(self.md5sum.lock().unwrap().clone()),
         }
     }
 }
@@ -1667,6 +1667,8 @@ impl From<KmerMinHashBTree> for KmerMinHash {
         new_mh.mins = mins;
         new_mh.abunds = abunds;
 
+        new_mh.md5sum = other.md5sum;
+
         new_mh
     }
 }
@@ -1691,6 +1693,8 @@ impl From<&KmerMinHashBTree> for KmerMinHash {
         new_mh.mins = mins;
         new_mh.abunds = abunds;
 
+        new_mh.md5sum = Mutex::new(other.md5sum.lock().unwrap().clone());
+
         new_mh
     }
 }
@@ -1714,6 +1718,8 @@ impl From<KmerMinHash> for KmerMinHashBTree {
         new_mh.mins = mins;
         new_mh.abunds = abunds;
 
+        new_mh.md5sum = other.md5sum;
+
         new_mh
     }
 }