Parham Kazemi

Closing due to being related to ntHash's old codebase. New ports are welcome!

Sure! Thank you @rchikhi.

Hi @shenwei356 and thanks for your comments! This was a known issue with the early versions of ntHash (see [this issue](https://github.com/bcgsc/ntHash/issues/7) and also ntHash2's paper). We've fixed the periodicity problem...

Apparently, SWIG (the tool used in btllib to generate Python wrappers) [also supports Go](https://www.swig.org/Doc3.0/Go.html). I'll try it out on ntHash and post any updates here.

Is there an easier way of doing this for a single SeqReader which reads sequences one by one? Like, after each iteration in the code below, can we have the...

Good idea, I like it. Is it something we'd like to implement directly in ntHash though? I think it can be a separate class (as a template, e.g., PhredFilter\) that...

> Are you thinking of template because of seednthash? Yeah basically I was thinking that since we have four ntHash classes, it'd be unusual to have a new feature implemented...

> > > Are you thinking of template because of seednthash? > > > > > > Yeah basically I was thinking that since we have four ntHash classes, it'd...