-
Notifications
You must be signed in to change notification settings - Fork 75
Add sum model #364
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
base: main
Are you sure you want to change the base?
Add sum model #364
Conversation
…operty to TPL models
…re derived from correlation not cor
…into add_sum_model
@LSchueler this is ready for review. |
Awesome, I'll need some time to thoroughly go throw your changes. |
First come, first serve :-D |
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.
Sorry for taking so long to finish this review, but as you know, I had the best reasons ;-)
I'm really looking forward to sum models being integrated into GSTools, but I found quite a few things which should be addressed before merging.
@LSchueler fixed (almost) all your complains. Second look? 🙃 |
This PR adds a
SumModel
class to represent sums of Covariance models.Sum-Model
SumModel
class+
operator:model = m1 + m2
m1 = model[0]
model[0].nu == model.nu_0
len_scale
is fixed, none of thelen_scale_<i>
can be fixed since len_scale is calculated from variance ratioszero_var
andmodel
attributes to Generator ABC to shortcut field generation for pure nugget modelsOther changes
var_raw
attribute from CovModel (was rarely used and only relevant for the truncated power law models)intensity
attribute which calculates whatvar_raw
was beforevar_raw
was a bad idea in the first place)