buzsakilab / buzcode

Code for internal lab sharing - polishing has started but is by no means complete
http://www.buzsakilab.com/
GNU General Public License v3.0
119 stars 128 forks source link

SleepScoreMaster update: use Power Spectrum Slope #265

Closed dlevenstein closed 5 years ago

dlevenstein commented 5 years ago

This is an update to the SleepScoreMaster algorithm for NREM detection - it adds the ability to use the slope of the (log-log) power spectrum (i.e. the 1/f exponent). So far I've found this to be more reliable in non-frontal cortex recordings and with noise, so I made it default (the previous weight method is still also available with SWWeights), but am hoping some people will try it out on their data as well.

Repull #262

brendonw1 commented 5 years ago

Dumb question - I use my own fork. How do I get this pull request into my own fork to try it? Or do I have to use the main fork?

dlevenstein commented 5 years ago

Hm - let's pull it into dev and you can check it there? If you want to try it before pulling, you might be able to open a pull request from my fork

brendonw1 commented 5 years ago

I'm fine with it in Dev. But I'm also more cavalier than some

On Mon, Oct 29, 2018 at 12:27 PM Dan Levenstein notifications@github.com wrote:

Hm - let's pull it into dev and you can check it there? If you want to try it before pulling, you might be able to open a pull request from my fork

— You are receiving this because your review was requested. Reply to this email directly, view it on GitHub https://github.com/buzsakilab/buzcode/pull/265#issuecomment-433976463, or mute the thread https://github.com/notifications/unsubscribe-auth/ADXrTforaZcDObWXl90WhnJa6_juHhSCks5upyxxgaJpZM4X_e5j .

brendonw1 commented 5 years ago

Oh yeah I can do from your fork to mine!. Will do that.

On Mon, Oct 29, 2018 at 12:28 PM Brendon Watson brendon.watson@gmail.com wrote:

I'm fine with it in Dev. But I'm also more cavalier than some

On Mon, Oct 29, 2018 at 12:27 PM Dan Levenstein notifications@github.com wrote:

Hm - let's pull it into dev and you can check it there? If you want to try it before pulling, you might be able to open a pull request from my fork

— You are receiving this because your review was requested. Reply to this email directly, view it on GitHub https://github.com/buzsakilab/buzcode/pull/265#issuecomment-433976463, or mute the thread https://github.com/notifications/unsubscribe-auth/ADXrTforaZcDObWXl90WhnJa6_juHhSCks5upyxxgaJpZM4X_e5j .

dlevenstein commented 5 years ago

Merging. If we find PSS to be worse than SWWeights after people have tried it out, we can change the default back to SWWeights.