Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Severe service deterioration for 2.6.* topology models #33

Open
mmarras opened this issue Mar 19, 2024 · 4 comments
Open

Severe service deterioration for 2.6.* topology models #33

mmarras opened this issue Mar 19, 2024 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@mmarras
Copy link

mmarras commented Mar 19, 2024

I don't know what changes induced this precisely, but anything >v.2.54 (I cannot test at 2.5.5, due to #27) leads to very poor performace of the topology variant.

from findpeaks import findpeaks

#smth = your dummy data here

firstn = 1
fp = findpeaks(method="topology", verbose=0)
df_peak = fp.fit(smth)["df"]
df_peak[
        (df_peak["rank"] != 0) & (df_peak["score"] != 0.0) & (df_peak["valley"] == False)  # noqa: E712
    ].sort_values("rank")[0:firstn]

Picture perferct results out of the box with 2.5.4, green X at positions of first peak:

image
image

Garbage with 2.6.0+ (2.5.5 not testable):
image
image

@erdogant
Copy link
Owner

Looks not so good indeed. Not sure what exactly changed but if you can share your toy example, I will look into it

@mmarras
Copy link
Author

mmarras commented Mar 28, 2024

I cannot give you the actual raw data, but I will generate a dataset that shows the effect after Easter.

@erdogant
Copy link
Owner

I updated to latest scipy and peakdetect again. Maybe this improves your results.
update with pip install -U findpeaks

@mmarras
Copy link
Author

mmarras commented May 17, 2024 via email

@erdogant erdogant added the bug Something isn't working label May 17, 2024
@erdogant erdogant self-assigned this May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants