Skip to content

noaa-nclimgrid example notebook#215

Merged
TomAugspurger merged 3 commits intomicrosoft:mainfrom
pholleway:feature/pah/notebook
Sep 19, 2022
Merged

noaa-nclimgrid example notebook#215
TomAugspurger merged 3 commits intomicrosoft:mainfrom
pholleway:feature/pah/notebook

Conversation

@pholleway
Copy link
Contributor

  • Added example notebook for noaa nclimgrid
  • API will need to be updated to production

@TomAugspurger
Copy link

Thanks!

I'm seeing some strange data when I run this on the (staging) Hub. Here's the first plot.

image

Where did you run this? I'm wondering if it could be a version issue.

@pholleway
Copy link
Contributor Author

pholleway commented Sep 13, 2022

Thanks!

I'm seeing some strange data when I run this on the (staging) Hub. Here's the first plot.

image

Where did you run this? I'm wondering if it could be a version issue.

@TomAugspurger That's odd! The data displays correctly for me running locally and on the hub (production). Pete mentioned it could be related to versioning of rasterio (he recommends rasterio 1.3.2) or odc (there was a recent release that may have caused issues). Here are the versions in production: image

@pholleway
Copy link
Contributor Author

@TomAugspurger Note: There was a bug introduced in rasterio 1.3.0 that may be related. rasterio/rasterio#2511

@TomAugspurger
Copy link

Yep, that's probably it. I'm on 1.3.0. I'm planning to update the container images this week anyway.

@TomAugspurger
Copy link

TomAugspurger commented Sep 19, 2022

Thanks @pholleway, things are working with rasterio>=1.3.0.

I pushed a small change to avoid re-using the plt variable name.

@TomAugspurger TomAugspurger merged commit 1a50bce into microsoft:main Sep 19, 2022
@pholleway
Copy link
Contributor Author

Sounds good. Thanks, @TomAugspurger.

@pholleway pholleway deleted the feature/pah/notebook branch October 13, 2022 15:59
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