search

geco-bern / agds

Applied Geodata Science book. Developed for the lecture(s) with the same name at the Institute of Geography, University of Bern.
https://geco-bern.github.io/agds/
Other
5 stars 6 forks source link

Adding first version of iML chapter #146

Closed padasch closed 1 year ago

padasch commented 1 year ago

Not sure if this is directly compatible with the only rendering because I updated renv with new packages and bumped some package versions (and R version). @khufkens maybe you can spot potential errors here?

EDIT: probably should set up rendering from my own fork instead of rising concerns like this 🙃

khufkens commented 1 year ago

Runners run on R 4.3.1

khufkens commented 1 year ago

@padasch I'm missing the vip package

padasch commented 1 year ago

Thanks for noticing, that slipped through. I get this error now and trying to figure out why I cannot install {vip} via {renv}:

> renv::install("vip")
Retrieving 'https://cran.rstudio.com/bin/macosx/big-sur-arm64/contrib/4.3/vip_0.3.2.tgz' ...
    ERROR [curl: (22) The requested URL returned error: 404]
Error: error downloading 'https://cran.rstudio.com/bin/macosx/big-sur-arm64/contrib/4.3/vip_0.3.2.tgz' [curl: (22) The requested URL returned error: 404]
khufkens commented 1 year ago

The vip package was deprecated (yesterday) as the maintainer forgot / does not want / can't address the issues raised by CRAN.

Package report: https://cran.r-project.org/web/packages/vip/index.html

Offending check results: https://cran-archive.r-project.org/web/checks/2023/2023-08-14_check_results_vip.html

Might be addressed soonish. https://github.com/koalaverse/vip/issues/153

padasch commented 1 year ago

Wow, what a timing... Alright, let's see if this is resolved soon. If needed, I can rewrite the code with the {iml} but I have not used it before and don't know its peculiarities. Could follow this: https://uc-r.github.io/iml-pkg#vip

Referring here issue #143 to keep tabs

khufkens commented 1 year ago

Just wait until the end of the week, should be resolved soon I think.

padasch commented 1 year ago

@khufkens, this problem should be resolved now, worked with runners on my fork.