Skip to content

Add ESIP July 2025 tutorial notebook#8

Open
omshinde wants to merge 5 commits intomainfrom
esip-july-2025
Open

Add ESIP July 2025 tutorial notebook#8
omshinde wants to merge 5 commits intomainfrom
esip-july-2025

Conversation

@omshinde
Copy link
Copy Markdown
Contributor

@omshinde omshinde requested a review from wildintellect July 22, 2025 22:06
@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@@ -0,0 +1,2212 @@
{
Copy link
Copy Markdown
Contributor

@wildintellect wildintellect Jul 22, 2025

Choose a reason for hiding this comment

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

Line #6.    # COPC_URL = "https://maap-ops-workspace.s3.amazonaws.com/shared/omshinde23/GLLIDARPC_FL_20200311_FIA8_l0s30.copc.laz?AWSAccessKeyId=ASIA43WBHR7NZHDNQVZK&Signature=TPJrtQdgnN7B0SlPMcO5fj%2BYlUk%3D&x-amz-security-token=IQoJb3JpZ2luX2VjENz%2F%2F%2F%2F%2F%2F%2F%2F%2F%2FwEaCXVzLXdlc3QtMiJGMEQCIEt17nYGesIjB2Lt6Zh2%2BQXL0nYznGJap04zu0PQ5Xl9AiAnaCcNGP0Gr1O%2Busr7BtPBxpluWp%2BdmyEG9ohFWxOxbSrBBQj1%2F%2F%2F%2F%2F%2F%2F%2F%2F%2F8BEAMaDDg4NDA5NDc2NzA2NyIMeZUz6%2FQ6JsH2bMpJKpUFjZTM6yuINXGefXwkPvebRDxO%2Fnr8rE1N2AMeyBhpBCthcpdDDAU8h4DuC6KKKkHkyz4%2BOOSC%2Fxm9uHi8NS4mfMXyKdJ1qmbFfKpvSnQ9ioAJGWZqGjImpkM5i4lNdWXBBf3ZoLnFN0H2D24EMeVjjYjg1LC4MC%2BSiH3wsGFwulocUq2IKlR0D9o76zsQ1bU%2Fuq3r4Oj3TzP3Ywa%2F3YpMLbE8E3IsFgWX4SMiYo%2BmDkUh8EAY0wD51vcM9t1iZ7TPXVT2cQBG3lVPJZMFwEN7mgFAgfvmVnMzcv2%2BNlW2hmsJkrSXvcaEC0IiGmzIlcgFgG1TXpRzjmJ64X%2BsFg%2F4molV5Kyp4hlLeMEdaKnwAdWHaFkbwEDoisQRlGvBUh8DVEIPaZYqPCCdT0db3YRhlSQNvCwQOYC7iGDABmwTIAJuEkmWGsfBIEnK7RHwHBnbcj%2Big0fzoYoFNwDHruo%2BSZe8LZ2TxW%2FkblIEBUtnRunE7MlVPDNR7TieFFV%2FtHHiVFDqqVVqS5%2Bxs%2FLEAwLKxbTr8Js%2BniFuOH2Te%2Fw9bqJA7DuJrmZ0ioYq2VRbWhe9bR0L9jj7jV6Sd%2BDe%2F3l%2FRw%2BJiSY0kyB%2BJzMFDR9c%2Bbc%2BqyrgouC0t6erAgpkY3rjqJY2MhHRIQiGJX9Lpzs1VGb4NcGcS2lnY0o6%2FpP21004ZyKdSfXDS1onu2GH9GxI2cdMFsfed%2FyColavsQyQTEKkqKiG6ujAXJz%2B5UMQPXAV0VP%2Bbixfuaa3Ghm5C%2FlXhu%2BlJEYye6xnwYydvIwkLcCQo1PP5ssd8FFMMJE4iZs7aYRzupwDBhB7mnNXEEfkXIYOmfcFKOQyJcGCgdx4vq7WWPnKr6vqd1tG4uKOLHH3xtr2UzDF0P%2FDBjqyAYKHl38UJKNtkzrZmTxpY2h8H9P6RXfgRP51bRNXz8mJ0eWR2KpBXEU%2F5lhepaKBjd1fKgYm1YP9p8joL0ARnGarrCN%2BVZDyi7U%2BbyTOek7vZDbDkUvsHiQ38yWFGxG%2FBEvGHaQh4kNxICZBU%2B62O38VLLTGzZTe%2Fp8UeEtsTU0F2KrVqGmYDXI%2B%2B%2Fu0FihytgG%2BTO2OgyUh9aeXYes26qqwzlCteqzJ6le6vZhGtdyT9is%3D&Expires=1753820169"

FYI on VEDA Hub I believe the MAAP bucket is accessible without a signed url.


Reply via ReviewNB

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I was wrong, MAAP bucket is only readable on MAAP Hub which doesn't have Binder set up. We could put a copy of the file in the nasa-veda-scratch bucket.

@@ -0,0 +1,2212 @@
{
Copy link
Copy Markdown
Contributor

@wildintellect wildintellect Jul 22, 2025

Choose a reason for hiding this comment

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

Link is incorrect


Reply via ReviewNB

@wildintellect
Copy link
Copy Markdown
Contributor

It's probably worth mentioning the LAZ file is so much smaller than LAS (18 MB vs 65 MB)

@wildintellect
Copy link
Copy Markdown
Contributor

@omshinde
Copy link
Copy Markdown
Contributor Author

@wildintellect I added new benchmarking experiments in part 2. Requesting review. Hopefully it will be directly reflected in the binder link. Thank you!

@omshinde omshinde requested a review from wildintellect July 23, 2025 04:35
@@ -0,0 +1,2485 @@
{
Copy link
Copy Markdown
Contributor

@wildintellect wildintellect Jul 23, 2025

Choose a reason for hiding this comment

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

Was there a bug that required Fiona to be used instead of pyogrio?


Reply via ReviewNB

@wildintellect
Copy link
Copy Markdown
Contributor

Minor note for later, when I built a custom image for this (not binder), I think I ended up with a different version of something, maybe earthaccess or python where the las_filename ended up being a PosixPath not a string, and it started causing problems unless you cast to str(las_filename). Lets not mess with this now since it's working with Binder.

@@ -0,0 +1,2485 @@
{
Copy link
Copy Markdown
Contributor

@wildintellect wildintellect Jul 23, 2025

Choose a reason for hiding this comment

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

You renamed this notebook.


Reply via ReviewNB

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

the Lonboard visualization isn't rendered in the NotebookSharing link


Click the badge below to launch the tutorial directly in your browser:

[![Binder](https://mybinder.org/badge_logo.svg)](https://mybinder.org/v2/gh/USERNAME/REPONAME/HEAD?filepath=cloud_optimized_lidar_full_merged.ipynb)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Did you test on mybinder? should we be using VEDA binder instead?

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.

2 participants