gpstuff-dev / gpstuff

GPstuff - Gaussian process models for Bayesian analysis
http://research.cs.aalto.fi/pml/software/gpstuff/
GNU General Public License v3.0
169 stars 61 forks source link

Added derivatives of Matern-5/2. #38

Closed opkoisti closed 6 years ago

opkoisti commented 6 years ago

I could not reproduce these errors for some reason. However, I corrected one mistake on line 409 in subfunction "trvar", where "selectedVariables" is skipped because of a typo. I hope this solves the problem you found.

In addition, I added "selectedVariables" functionality also in subfunction "ginput". It was missing also in Matern-3/2 already before adding the derivatives, so I suspect that this might not have affected on your tests.

jpvanhat commented 6 years ago

Everything seems to work now. I will approve the pull request.