Aasiya3 / ElephantBook_FieldGuide

Apache License 2.0
1 stars 1 forks source link

Update DatabaseWrapper::seekDistance #46

Open bsakai2000 opened 2 years ago

bsakai2000 commented 2 years ago

SeekDistance crashes on 0-length string. We need to either

  1. Update seekDistance to make it not crash
  2. Add some sort of normalizer to make sure SEEK data conforms

Here's one option which seems to work OK, but idk if this conforms to our hamming distance spec. @Aasiya3 since you wrote the original code for #10 I'll defer to you here. If you don't like this, we can close this and open and assign an issue