Skip to content

Conversation

@acblackford
Copy link
Contributor

@acblackford acblackford commented Dec 9, 2025

Description

This notebook shows a tutorial on how to use Folium to create an interactive mapping experience similar to the VEDA environment.

What type of example is this?

Tutorial

Checklist

The following checklist ensures that our notebooks are internally consistent (read more)

  • The first cell contains the rendering information with author and data
  • The notebook has a Run this Notebook section (with filename in link)
  • The notebook has an Approach section
  • All the packages that are imported in the notebooks are used within the notebook
  • Any complex geometries are accessed from remote storage
  • Each code cell comes after a markdown cell containing explantory text
  • All cells in the notebook book have been run in order with a fresh kernel (cell numbers should be 1, 2, 3, 4, ...)
  • If the example type is Dataset the filename is <collection_id>.ipynb

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@github-actions
Copy link

github-actions bot commented Dec 9, 2025

PR Preview Action v1.6.3

🚀 View preview at
https://NASA-IMPACT.github.io/veda-docs/pr-preview/pr-263/

Built to branch gh-pages at 2025-12-09 20:50 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@acblackford acblackford changed the title Tutorial Notebook: Folium Mapping [Notebook] Tutorial - Folium Mapping Dec 10, 2025
@@ -0,0 +1,1613 @@
{
Copy link

@tylanderson tylanderson Dec 11, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line #5.    import plotutils as putils

issue: currently this is not included alongside and will error on import.

I might suggest that if this is meant to be an exploration of folium in general with VEDA, that you not rely on it and instead pull relevant sections that demonstrate how to use folium along side VEDA into the notebook (will be longer but expose users to the functions if they would like to modify)


Reply via ReviewNB

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tylanderson plotutils is a notebook organization tactic that @freitagb has already approved for prior notebooks built to support Data Stories. Would plotutils not be able to be used in tutorial notebooks as well?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did see that 2024 tornadoes notebook uses plotutils, as a sidecar, however that is not in the same folder as this notebook or installed as a package, so the immediate issue is that the import will not resolve and this first cell will error

@@ -0,0 +1,1613 @@
{
Copy link

@tylanderson tylanderson Dec 11, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line #8.    items = list(results.items())

suggestion: You could change this to get the first item if that's all that's needed with next(results.items(), None)


Reply via ReviewNB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants