Skip to content
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

update python to latest #49

Closed
wants to merge 3 commits into from
Closed

update python to latest #49

wants to merge 3 commits into from

Conversation

ngam
Copy link
Contributor

@ngam ngam commented Aug 10, 2022

No description provided.

@ngam ngam mentioned this pull request Aug 10, 2022
4 tasks
Copy link
Contributor

@TomAugspurger TomAugspurger left a comment

Choose a reason for hiding this comment

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

Thanks for this. I was hoping to wait on upgrading the Python version until we could get a binder set up or jupyterhub/kubespawner#279 implemented, but I haven't had time to work on either of those.

I do have one request about keeping pystac-client pinned for now, since its 0.4.0 release broke some workflows.

@@ -70,7 +70,7 @@ dependencies:
- pydap
- pyepsg
- pygeos
- pystac-client==0.3.5 # https://github.com/stac-utils/pystac-client/issues/237
- pystac-client>=0.3.5 # https://github.com/stac-utils/pystac-client/issues/237
Copy link
Contributor

Choose a reason for hiding this comment

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

This will need to be ==3.5 until the next release (0.5.0 I think), which will include the fixes for stac-utils/pystac-client#237.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay! Sorry that was too aggressive of a change (I simply did changed == to >= naively)

Let's continue in #48

@ngam
Copy link
Contributor Author

ngam commented Aug 15, 2022

continue in #48

@ngam ngam closed this Aug 15, 2022
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.

None yet

2 participants