valhalla / docker

docker
MIT License
17 stars 22 forks source link

merged associate segs into cut_tiles #22

Closed gknisely closed 7 years ago

gknisely commented 7 years ago

@heffergm please review when you have a minute

heffergm commented 7 years ago

I gather that there are three or so S3 variables that are required to be set in the env but aren't provided with defaults at the head of the file? If I've got that right, maybe we could just mention them in comments somewhere up top?

Otherwise, let's :ship: :it: and we can get it running next week.

heffergm commented 7 years ago

👍