OpenDroneMap / FIELDimageR

FIELDimageR: A R package to analyze orthomosaic images from agricultural field trials. This package is a compilation of functions to analyze pos-mosaicking images from research fields, and allows to: crop the image; remove soil effect; build vegetation indices; rotate the image; build the plot shapefile; extract information for each plot; and evaluate stand count, canopy percentage, and plant height.
GNU General Public License v2.0
237 stars 89 forks source link

Add a Dockerfile to build docker images #7

Closed Chris-Schnaufer closed 3 years ago

Chris-Schnaufer commented 4 years ago

Building Docker images with the base requirements met would allow others to use FIELDimageR with less startup overhead.

I created a Dockerfile at https://github.com/Chris-Schnaufer/FIELDimageR/blob/add_dockerfile/Dockerfile that can be merged (if desired) and used to build and distribute a Docker image.

dlebauer commented 4 years ago

A few thoughts:

FROM rocker/rstudio:3.6.3
MAINTAINER="Chris Schnaufer <schnaufer@email.arizona.edu>"

RUN apt-get update && apt-get install -y --no-install-recommends \
    libxml2-dev 
    zlib1g-dev 
    libfftw3-dev 
    gdal-bin 
    libgdal-dev 
    libxt-dev

RUN install2.r --error \
    sp
    raster
    rgdal
    scales
    xml2
    git2r
    usethis
    fftwtools
    devtools

RUN installGithub.r filipematias23/FIELDimageR

In addition, If you start from rocker/geospatial you get a lot of these dependencies already built in (including tidyverse and a lot of geospatial utilities - not just those required by FieldImageR but some that could be useful for running downstream analyses, as well as a lot of bloat!!)

filipematias23 commented 4 years ago

Thank you for sharing ..!!!

Chris-Schnaufer commented 4 years ago

I have updated the Dockerfile to take advantage of suggested installation changes.

I am putting the suggested additions for downstream analysis into another issue: #9

Chris-Schnaufer commented 4 years ago

Linking to https://github.com/OpenDroneMap/FIELDimageR/pull/10

dlebauer commented 3 years ago

this can be closed since #10 has been merged and suggestions for improvement are in #9

filipematias23 commented 3 years ago

Hi @Chris-Schnaufer and @dlebauer, Thank you a lot for helping with this project. I am wondering what is the next step? How can I integrate your suggestions (#9 and #10) to the tutorial?

All the best, Filipe

Chris-Schnaufer commented 3 years ago

I am wondering what is the next step? How can I integrate your suggestions (#9 and #10) to the tutorial?

I think the next step as described in #9 would be to determine if those are changes that are desired and, if so, update the Dockerfile to use the new base image. I don't know who would be best suited to determine if those changes are desired