Nima Sarajpoor

@seanlaw Please feel free to check out the last test function I added to the snippet unit test. As you can see, the test fails.

@seanlaw > Have you been able to confirm the root cause for the difference? So, after some investigation, I noticed that both naive and performant version have close values in...

> which one is actually "off"? Is it the naive version or the performant version? It is the performant version. To be more precise, the distance profile between query `T[40:43]`...

@seanlaw > > which one is actually "off"? Is it the naive version or the performant version? > > It is the performant version. To be more precise, the distance...

> It should be this, right: > > ``` > if pearson > config.STUMPY_CORRELATION_THRESHOLD: > pearson = 1.0 > ``` Correct. It can now resolve the following issue as well...

> @NimaSarajpoor would you like to review as well? Yes, I will review it soon :)

I am planning to work on this as I see it as a good opportunity to learn something new. Now, I need to handle an issue that is different than...

@rasbt Thanks for providing more info. That should help me a lot for sure. I will work on this and if I get into a trouble or if I have...

@rasbt Hi Sebastian... I just wanted to let you know that My mind is occupied with something and I cannot focus well. I will start working on this in about...

@rasbt I tried to test the existing code regarding this issue. I have a question for you: > I think the easiest way to address this issue is to introduce...