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

Change Burst Criteria #5

Open
giulioberna opened this issue May 27, 2014 · 2 comments
Open

Change Burst Criteria #5

giulioberna opened this issue May 27, 2014 · 2 comments

Comments

@giulioberna
Copy link

As previously discussed, I think it would be nice to add a step for detection of potential missing waves in a burst (e.g., lowering the detection crtieria by a 10-20%) for a window comprised between the supposed beginning and end of the burst.

Looking for missing waves at the beginning and at the end of the burst (ie, before first wave and after last wave) could be a bit more risky (so I would start with missing waves inside the detected bursts limits), but maybe we should try it at some point also.

@Mensen
Copy link
Owner

Mensen commented May 27, 2014

I agree that searching for a true beginning might be quite difficult considering we may lower the threshold too much and it would never stop finding waves... but an absolute minimum could nonetheless be specified, rather than just repeatedly dropping the amplitude criteria by 10% after a wave is found.

So something like:
if next wave is close in time... drop amplitude criteria by 10% to a limit of 30% of the original etc

@giulioberna
Copy link
Author

I think for most analyses, it would be good enough to find missing waves between the beginning and ending points already detected... for instance, if you want to calculate the density of waves in the burst.

The search for waves at the beginning and at the end is more risky and should probably be optional (the idea of a maximum drop in amplitude is good I think).

@Mensen Mensen added the ST label Jun 13, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants