nasaharvest / crop-mask

End-to-end workflow for generating high resolution cropland maps
Apache License 2.0
95 stars 28 forks source link

Dockerfile for GDAL and Python #70

Closed ahong8595 closed 2 years ago

ahong8595 commented 2 years ago

Dockerfile to install Python and GDAL.

ahong8595 commented 2 years ago

Can run gdal, google-cloud storage, and merge script. You would simply need to move the folder over to merge using the COPY command. If you want to use vim, use these commands in the container: apt-get update, apt-get install vim.

ivanzvonkov commented 2 years ago

@ahong8595 did you mean to merge this pull request into master?

ahong8595 commented 2 years ago

Sorry. Did not mean to pull to the master branch. Can you take a look at the Dockerfile I uploaded to make sure everything works for you?

ivanzvonkov commented 2 years ago

Sorry. Did not mean to pull to the master branch. Can you take a look at the Dockerfile I uploaded to make sure everything works for you?

Dockerfile looks fine! There are two though, one needs to be removed.

We have to be very careful about credentials though:

Since this code is on master I suggest making a new branch gdal-dockerfile-fix, making the changes there and creating a new PR.

ivanzvonkov commented 2 years ago

@ahong8595 did you see this?

ahong8595 commented 2 years ago

Yup. I removed the file and command from my Dockerfile and replaced it on my branch.

ivanzvonkov commented 2 years ago

What about a PR to make that change official?