<div dir="auto">Thanks for working on this. That sounds like a good plan to me. I should be able to review this week.<div dir="auto"><br><div dir="auto">Eric</div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Feb 14, 2018 7:11 AM, "Lars G." <<a href="mailto:lagru@mailbox.org">lagru@mailbox.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 26.01.2018 20:49, Lars G. wrote:<br>
> Dear SciPy devs,<br>
><br>
> I'd like to highlight my current pull request which extends SciPy's peak<br>
> finding capabilities in the signal module.<br>
> <a href="https://github.com/scipy/scipy/pull/8264" rel="noreferrer" target="_blank">https://github.com/scipy/<wbr>scipy/pull/8264</a><br>
> I've tried to address all feedback that was given (until now) in said<br>
> PR.<br>
> [...]<br>
<br>
I am quite confident with the current state and would encourage a last<br>
review. So feel free to chime in if you have some tweaks or refactoring<br>
to suggest.<br>
<br>
The only outstanding question is about the name of `find_peaks`. As the<br>
discussion on this hasn't really yielded an agreed upon solution I'm<br>
going to leave the name as it is for the time being.<br>
<br>
Going forward I'd like to build on this with new PRs:<br>
<br>
- Some parts (especially inside the big loops in `peak_prominences` and<br>
`peak_widths`) would profit from one or two private functions written in<br>
Cython. I get noticeable speed ups if I just replace a few steps of the<br>
algorithms with Cython. `find_peaks` would profit from this as well. I<br>
think this is worthwhile because prominence is a very useful parameter<br>
and it shouldn't be to much work (unit tests and PYX file already exist).<br>
<br>
- A tutorial that shows some more complicated use cases for the new<br>
functions and peak finding in general. If I feel confident, I may even<br>
give some examples and guidelines for `find_peaks_cwt` whose<br>
documentation was described as confusing in the past.<br>
<br>
Best regards,<br>
Lars<br>
______________________________<wbr>_________________<br>
SciPy-Dev mailing list<br>
<a href="mailto:SciPy-Dev@python.org">SciPy-Dev@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/scipy-dev" rel="noreferrer" target="_blank">https://mail.python.org/<wbr>mailman/listinfo/scipy-dev</a><br>
</blockquote></div></div>