shogun-toolbox / shogun-demo

Shogun Toolbox Interactive Web Demos
7 stars 17 forks source link

added a button for ML2 in gp demo. issue #2085 #35

Closed kislayabhi closed 10 years ago

karlnapf commented 10 years ago

Just to make sure: Does this smoothly work locally with the latest Shogun version? I cannot test here

kislayabhi commented 10 years ago

@karlnapf , yes, this is working fine with the current shogun version locally. But let me correct the problem you have addressed. Also, I will try to add some description.

kislayabhi commented 10 years ago

@karlnapf , I have updated the PR with this new commit. One thing I wanted to know: In gp notebook , while using ML2, you used : inf = ExactInferenceMethod(GaussianKernel(10, taus[i]), feats_train, ZeroMean(), labels_train, GaussianLikelihood()) .

Here the term taus[i] is 32. Will varying this term matter in learning the hyper parameters?

karlnapf commented 10 years ago

I dont really get the question. Do you mean from where the ML2 optimisation is started? I think its started from current values...

karlnapf commented 10 years ago

Nice. Cant wait to try this :)

karlnapf commented 10 years ago

Ok, finally some feedback on this one:

karlnapf commented 10 years ago

The ML2 button should be documented. Say what happens when this is clicked

karlnapf commented 10 years ago

The kernel scale should also be a parameter to set. This is m_scale in CExactInferenceMethod

karlnapf commented 10 years ago

When one presses the ML2 button, the parameters in the boxes should be updated also, not just the plot.

karlnapf commented 10 years ago

The ML2 button should also work for other kernels, currently it always reverts to the Gaussian one from what I can see from the results

karlnapf commented 10 years ago

Label of the button should be "Use ML2" (whitespace) button "clear" should be "Clear" capital

karlnapf commented 10 years ago

The grid that is used to evaluate the regression function should be more fine. Use a few more points

karlnapf commented 10 years ago

I suggest you add another button next to the train GP one, which is "Show predictive", which creates heatmaps in the same style as the notebook (ask me if unclear) Those are just a 1D Gaussian centered at the mean prediction with variance given by the GP

karlnapf commented 10 years ago

Sometimes the ML2 fails and just produces a constant prediction, could you investigate why what is?

kislayabhi commented 10 years ago

@karlnapf , Yeah. Will make sure its done once I get some respite from my current work.

karlnapf commented 10 years ago

@Saurabh7 could you maybe take care of this?

karlnapf commented 10 years ago

@kislayabhi is busy with his own stuff and this is more part of your project now :)

karlnapf commented 10 years ago

Some of the above comments were already addressed in the GP classification demo btw

karlnapf commented 10 years ago

Suggestion: Have a dropdown menu for parameter selection as in GPClassify

karlnapf commented 10 years ago

Suggestion: Have a FITC sparse GP regression option where one can put the inducing points by hand (right click) see the notebook for how this works

Saurabh7 commented 10 years ago

For reference, will work on this.

Saurabh7 commented 10 years ago

I couldn't move the data section after the feature one since I am using the loaded data file to demonstrate feature methods and column major format. oops