search

baccuslab / pyret

Python tools for analysis of neurophysiology data
https://pyret.readthedocs.io/en/master/
MIT License
35 stars 8 forks source link

Adds a gaussian process (GP) regression nonlinearity. #71

Closed nirum closed 8 years ago

nirum commented 8 years ago

This adds a GP regression model for nonlinearities.

It uses the GPy library to fit GP Regression models, which allows us to model both a mean nonlinearity and uncertainty (variance) around the nonlinearity.

nirum commented 8 years ago

@bnaecker is it cool if I merge?

nirum commented 8 years ago

screen shot 2016-09-09 at 5 13 14 pm

nirum commented 8 years ago

^ example of a nonlinearity fit using Binterp vs. GaussianProcess

bnaecker commented 8 years ago

This looks good to me. I think just update the requirements.txt to make sure setup still works fine, and I'm good with merging.

nirum commented 8 years ago

The GPy is not technically required, it will throw an error if you don't have it. I could add it to requirements.txt and make it a dependency, thoughts?

On Sep 9, 2016, at 6:49 PM, bnaecker notifications@github.com<mailto:notifications@github.com> wrote:

This looks good to me. I think just update the requirements.txt to make sure setup still works fine, and I'm good with merging.

You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/baccuslab/pyret/pull/71#issuecomment-246080756, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AA3OlntNkf1SJ4Ut8icGXFLAI-K9BECZks5qogymgaJpZM4I5SEF.

bnaecker commented 8 years ago

Ah, I see. I think it’s fine to keep as-is for now.

On 09 Sep 2016, at 18:53, Niru Maheswaranathan notifications@github.com wrote:

The GPy is not technically required, it will throw an error if you don't have it. I could add it to requirements.txt and make it a dependency, thoughts?

On Sep 9, 2016, at 6:49 PM, bnaecker notifications@github.com<mailto:notifications@github.com> wrote:

This looks good to me. I think just update the requirements.txt to make sure setup still works fine, and I'm good with merging.

You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/baccuslab/pyret/pull/71#issuecomment-246080756, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AA3OlntNkf1SJ4Ut8icGXFLAI-K9BECZks5qogymgaJpZM4I5SEF. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/baccuslab/pyret/pull/71#issuecomment-246080997, or mute the thread https://github.com/notifications/unsubscribe-auth/ADG64xDdDH3mzLuGLsGFN5ovwOOdaZNNks5qog2ngaJpZM4I5SEF.