search

ProjectPythia / landsat-ml-cookbook

Machine learning on Landsat satellite data using open source tools
https://projectpythia.org/landsat-ml-cookbook/
Apache License 2.0
11 stars 4 forks source link

Build notebooks with binder #6

Closed r-ford closed 1 year ago

r-ford commented 1 year ago

@droumis These are some great notebooks, thanks for writing them up! With this PR, I'm trying to get the notebooks building on our BinderHub. Once it is working, we can get it in the Cookbook gallery so others can easily find it.

github-actions[bot] commented 1 year ago

👋 Thanks for opening this PR! The Cookbook will be automatically built with GitHub Actions. To see the status of your deployment, click below. 🔍 Git commit SHA: 87db72fcefe58b065959ba51d818ceacdfa980cc ✅ Deployment Preview URL: https://ProjectPythia.github.io/landsat-ml-cookbook/_preview/6

r-ford commented 1 year ago

Building on the BinderHub did not work because the local data used in the notebooks could not be accessed, but the notebooks can build just fine on GitHub. In the future maybe we can add the example data to pythia-datasets or figure out some other way to run them on the BinderHub. I also fixed a few config settings and typos, so I think this is ready to be added to the gallery.

droumis commented 1 year ago

@r-ford, thanks so much for the help with the config settings, typos, and feedback! I'm going to spend some time now trying to change the data access approach to be more cloud-oriented from the start. Hopefully, that will work around the local data access issues.