-
Notifications
You must be signed in to change notification settings - Fork 310
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Data API Tile Preview Demo #350
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Ash, this looks really great and will be a helpful add to this repository. Looks good to me!
this looks great, just a few comments:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's great to see this.
One interesting thing about this demo is that it leverages two different APIs (Data API and Tile Service API), both of which are now maintained by the same team, which is tasked with the "Gather" phase of the platform experience. It might be worth (if not now, then at some point) renaming the directory to reflect that customer journey stage rather than being specific to Data API. @matthew-ballard, do you have thoughts on this?
Also best to keep the README up-to-date. |
Documentation updates Co-authored-by: Nik (Anna) Bladey <[email protected]>
@matthew-ballard thanks for the suggestion. This seems like a topic for bigger conversation outside of this specific notebook. Including the special notebook-specific dependencies inline ensures that they don't get lost if someone shares or runs just the isolated notebook (without the full repo context). For example, some users will run individual notebooks directly in colab. IMO we should strive to make each notebook fully runnable (or documented) in a standalone environment. It is also a common pattern in the data science community, for example see:
Are you recommending updating the requirements.txt in I would also like to keep the install command for those who wish to use this notebook without dealing with the global docker env setup so that it can work in colab or other environments with ease. |
Add Demo Notebook for Rendering Data API Search Results as Preview Tile Layers
This PR adds a demo notebook that showcases how to visualize Planet’s Data API search results as preview tile layers, similar to the preview experience in Planet Explorer, but directly in Python.
📋 Key Features:
🛠 What’s Included:
🌟 Value:
See #349