-
Notifications
You must be signed in to change notification settings - Fork 74
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Krige unification #97
Conversation
…inverse / unbiased switch / use covariance
I still need to update the call signatures of the different kriging methods to include the |
…d pseudo inverse selector
Done. @LSchueler you can start your review ;-) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As I already said in PR #95, great work!
Just a typo hint and a suggestion to think about. Functions are first class objects in Python, hurray!!
Closes #79
Fixes #89
Here comes the great unification of our
krige
submodule:Krige
class now provides everything in one placeonly_mean
in the call routineSimple
/Ordinary
/Universal
/ExtDrift
/Detrended
are only shortcuts toKrige
with limited input parameter listcovariance
function to build up the kriging matrixunbiased
switch was added to enable simple kriging (where the unbiased condition is not given)exact
switch was added to allow smother results, if anugget
is present in the modelcond_err
parameter was added, where measurement error variances can be given for each conditional pointpseudo_inv
), this is equal to solving the system with least-squares and prevents numerical errorsPS: I needed to create a new PR, since there were some commits in the branch, that weren't recognized by the last PR.