firedpy icon indicating copy to clipboard operation
firedpy copied to clipboard
verified publisher icon earthlab

Issue 89

Open Ckster opened this issue 2 years ago • 3 comments

I updated the URLs in functions and main to https://e4ftl01.cr.usgs.gov/MOTA/MCD12Q1.061/. Dockerfile has base image updated and environment.yml has versions added to the packages. I pushed a new Docker image to earthlabcu/firedpy:issue_89 and tested it out on Samoa. If it checks out with @admahood , I'll change the tag to latest.

Ckster avatar Sep 13 '23 21:09 Ckster

MCD12Q1 is not a burned area product, it's a landcover product. If the landcover product we're using is also out of date we should also change that though

On Wed, Sep 13, 2023 at 3:05 PM Erick Verleye @.***> wrote:

I updated the URLs in functions and main to https://e4ftl01.cr.usgs.gov/MOTA/MCD12Q1.061/. Dockerfile has base image updated and environment.yml has versions added to the packages. I pushed a new Docker image to earthlabcu/firedpy:issue_89 and tested it out on Samoa. If it checks out with @admahood https://github.com/admahood , I'll change the tag to latest.

You can view, comment on, or merge this pull request online at:

https://github.com/earthlab/firedpy/pull/90 Commit Summary

File Changes

(4 files https://github.com/earthlab/firedpy/pull/90/files)

Patch Links:

  • https://github.com/earthlab/firedpy/pull/90.patch
  • https://github.com/earthlab/firedpy/pull/90.diff

— Reply to this email directly, view it on GitHub https://github.com/earthlab/firedpy/pull/90, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABXI3P7IEDPBIDXUVZP2OBTX2INZ3ANCNFSM6AAAAAA4XDAKEQ . You are receiving this because you were mentioned.Message ID: @.***>

admahood avatar Sep 13 '23 21:09 admahood

But the existing path essentially just needs to have a .061 added to the end, essentially - check with Max Cook, he has used the new product recently

On Wed, Sep 13, 2023 at 3:10 PM Adam Mahood @.***> wrote:

MCD12Q1 is not a burned area product, it's a landcover product. If the landcover product we're using is also out of date we should also change that though

On Wed, Sep 13, 2023 at 3:05 PM Erick Verleye @.***> wrote:

I updated the URLs in functions and main to https://e4ftl01.cr.usgs.gov/MOTA/MCD12Q1.061/. Dockerfile has base image updated and environment.yml has versions added to the packages. I pushed a new Docker image to earthlabcu/firedpy:issue_89 and tested it out on Samoa. If it checks out with @admahood https://github.com/admahood , I'll change the tag to latest.

You can view, comment on, or merge this pull request online at:

https://github.com/earthlab/firedpy/pull/90 Commit Summary

File Changes

(4 files https://github.com/earthlab/firedpy/pull/90/files)

Patch Links:

  • https://github.com/earthlab/firedpy/pull/90.patch
  • https://github.com/earthlab/firedpy/pull/90.diff

— Reply to this email directly, view it on GitHub https://github.com/earthlab/firedpy/pull/90, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABXI3P7IEDPBIDXUVZP2OBTX2INZ3ANCNFSM6AAAAAA4XDAKEQ . You are receiving this because you were mentioned.Message ID: @.***>

admahood avatar Sep 13 '23 21:09 admahood

Ah OK, yeah the land cover has version 061 too. It looks like we are getting burned product from the UMD sftp server, not https://e4ftl01.cr.usgs.gov/MOTA/MCD64A1.061/ . Reading through their docs, we just need to update the 'C6' directory to be 'C61'. Just pushed with that update. Will reupload the docker now

Ckster avatar Sep 13 '23 21:09 Ckster