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
2 include new sampled data example #4
Conversation
…om/jakobdambon/varycoef into 2-include-new-sampled-data-example
I had a quick look. seems reasonable. I will have a closer, detailed look once I am back in CH.
…----- Original Message -----
From: "Jakob Dambon" ***@***.***>
To: "jakobdambon/varycoef" ***@***.***>
Cc: "reinhard" ***@***.***>, "Review requested" ***@***.***>
Sent: Tuesday, 7 June, 2022 23:23:56
Subject: Re: [jakobdambon/varycoef] 2 include new sampled data example (PR #4)
@jakobdambon requested your review on: jakobdambon/varycoef#4 2 include new
sampled data example.
--
Reply to this email directly or view it on GitHub:
#4 (comment)
You are receiving this because your review was requested.
Message ID: ***@***.***>
--
______________________________________________________________________
University of Zurich ***@***.***
Department of Mathematics tel: +41-44-63-55843
and Institute of Computational Science office: Y27 J34
CH-8057 Zurich https://www.math.uzh.ch/furrer
https://twitter.com/ReinhardFurrer
|
Good to go. 2 cents: For Would it make sense to include the wrapper functions |
I've added the I don't understand your comment on the example / figure / additiveness of the SVCs. For a nice example, I use a code like this:
Could you provide an example? For your final comment, I will put it in a backlog #5. |
I was thinking of:
It was helpful to understand the role of the covariate. I struggle with random covariates. Just a tiny detail, nothing to worry. |
Using the
spam::rmvnorm
suite to sample SVC data, therefore replacing the old functions depending on packageRandomFields
.Main changes:
SVCdata
, that has been sample from theRandomFields
package. In the help file, I cite the RandomFields JSS article to give them the appropriate credit.sample_SVCdata
.