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

init deprecation warnings for each scene that is searched with porder search #40

Closed
rbavery opened this issue Dec 10, 2019 · 2 comments
Closed

Comments

@rbavery
Copy link
Contributor

rbavery commented Dec 10, 2019

Hi Sam!

This looks to be because of the form pyproj.Proj(init=epsgcode) that is used in geojson2id.py. I took a crack at solving this since the warnings were filling up a notebook I was running and also updated the code to use at least pyproj 2.2.2, which comes with > 10x speedup for the search function (according to my own test run.) I'm submitting a PR soon

The warnings:

Running search for a maximum of: 1000000 assets
/home/rave/miniconda3/envs/porder-test/lib/python3.8/site-packages/pyproj-2.4.2.post1-py3.8-linux-x86_64.egg/pyproj/crs.py:77: FutureWarning: '+init=<authority>:<code>' syntax is deprecated. '<authority>:<code>' is the preferred initialization method.
  return _prepare_from_string(" ".join(pjargs))
/home/rave/miniconda3/envs/porder-test/lib/python3.8/site-packages/pyproj-2.4.2.post1-py3.8-linux-x86_64.egg/pyproj/crs.py:77: FutureWarning: '+init=<authority>:<code>' syntax is deprecated. '<authority>:<code>' is the preferred initialization method.
  return _prepare_from_string(" ".join(pjargs))
/home/rave/miniconda3/envs/porder-test/lib/python3.8/site-packages/pyproj-2.4.2.post1-py3.8-linux-x86_64.egg/pyproj/crs.py:77: FutureWarning: '+init=<authority>:<code>' syntax is deprecated. '<authority>:<code>' is the preferred initialization method.
  return _prepare_from_string(" ".join(pjargs))
...
@rbavery
Copy link
Contributor Author

rbavery commented Dec 10, 2019

The PR I'm submitting should update the pyProj requirement from 1.9 and I notice in the docs this link is suggested to install pyproj, and the wheels only go up to 1.9 (which was released back in 2016): https://www.lfd.uci.edu/~gohlke/pythonlibs/#pyproj

I've found conda an easier method for getting geospatial dependencies installed, what do you think about amending the windows and other installation instructions to suggest a conda environment as the main way to get setup with porder? I taught this tutorial at UCSB where we did just that with an environment.yml file and it went alright, easier than pip at least: https://github.com/rbavery/porder_tutorial

samapriya added a commit that referenced this issue Dec 14, 2019
- No more depecration or future warnings from pyproj [Issue 40](#40)
- `porder search` is faster
- Python 2.7 will reach the end of its life on January 1st, 2020. No more python 2 support from v0.5.9
@samapriya
Copy link
Collaborator

Fixed in issues 0.5.9 based on PR from @rbavery . Thanks Ryan. porder now supports python3 onwards only

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

No branches or pull requests

2 participants