MAAP-Project / stac_ipyleaflet

Apache License 2.0
29 stars 2 forks source link

Make env config driven file and pre-populate layers conditionally - Preparation for VEDA release #99

Closed sandrahoang686 closed 1 year ago

sandrahoang686 commented 1 year ago

Summary:

NOTE: A .env file will need to exist now and code should be deployed with the env vars defined

Fixes or Addresses Issue #: [ticket number & link]

https://github.com/NASA-IMPACT/veda-analytics/issues/82#issuecomment-1687139940

Checklist before requesting a review:

sandrahoang686 commented 1 year ago

@emmalu what the warning logs will look like

Screenshot 2023-08-29 at 1 00 48 PM Screenshot 2023-08-29 at 1 01 12 PM
emmalu commented 1 year ago

@emmalu what the warning logs will look like Screenshot 2023-08-29 at 1 00 48 PM Screenshot 2023-08-29 at 1 01 12 PM

Any chance we can align the second os.environ( statement? We may also want to be more specific and suggest re-starting the notebook, running the code snippet in a cell of its own, and then creating the stac_ipyleaflet map object again.

And in the case of the STAC_CATALOG_OBJECT, we may want to be more clear about setting the object as this dictionary:

os.environ['STAC_CATALOG'] = {
    "name": "REPLACE WITH STAC CATALOG NAME FOR LABEL USE"
    "url": "REPLACE WITH STAC CATALOG URL"
}