neuromodulation / py_neuromodulation

Real-time analysis of intracranial neurophysiology recordings.
https://neuromodulation.github.io/py_neuromodulation/
MIT License
42 stars 9 forks source link

Performance regression in MNE connectivity #344

Closed toni-neurosc closed 3 months ago

toni-neurosc commented 3 months ago

I was trying to get some performance metrics to present my contributions at the lab meeting next week, and I noticed that while most features are faster to calculate now than they were before, the opposite happened with MNE connecivity:

This is a tremendous performance regression so some kind of bug must have been introduced at some point, need to investigate.

toni-neurosc commented 3 months ago

Ok, I did the investigation and concluded that there was no regression, it was an artifact in the profiling results caused by the logging to the console that MNE was doing in prior versions (before I set it to verbose=False). It probably has to do with the specific profiler I'm using (yappi) or how i configured it (yappi.set_clock_type("cpu")).

I need to find a more reliable way to profile. Perhaps try other profilers: