Public-Health-Scotland / technical-docs

Technical documentation, including guidance and best practice for Public Health Scotland (PHS)
https://public-health-scotland.github.io/knowledge-base/docs
5 stars 2 forks source link

REQ - Simplify Geospatial install + updating #75

Open Moohan opened 10 months ago

Moohan commented 10 months ago

Is your feature request related to a problem? Please describe. The geospacial guidance is great but suggests installing specific versions of packages which may not be necessary. Related, there is no guidance on how to update packages (for those where we don't have to stick to a specific version.

Describe what you'd like Simplify the Geospacial guidance by:

  1. Check which packages can just install the latest versions and simplify the code accordingly. - I just tested raster and the latest version seems fine but with terra it isn't...
  2. Make the ncpus code optional. Rather than having Ncpus = ncpus in every install chunk, include options(Ncpus = as.numeric(parallelly::availableCores())) in guidance (could be added to Rprofile).
  3. Confirm that the additional configure.args and INSTALL_opts are strictly necessary for each install and remove them to simplify the code if not.

Add a section on updating geospacial packages. This might be: packages x and y can't be updated as you have to use version x.x but reuse the code for z to update it to the latest version...

Additional context I'd be happy to come back to this and do a PR at some point but don't have time to do the required testing just now!