-
Notifications
You must be signed in to change notification settings - Fork 9
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
added option for explicit rpm cluster CRS with default to WGS 84 #168
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
reVX/rpm/rpm_clusters.py
Outdated
crs="EPSG:5070") | ||
crs=source_crs) | ||
|
||
if target_crs is not None: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you know if Geopandas efficiently handles the case where the df crs == target crs? If we want to be extra cautious, we could change this to if target_crs != source_crs:
in order to be completely sure that no unnecessary computation is happening
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No idea haha that's a good idea to add that!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #168 +/- ##
==========================================
- Coverage 80.64% 80.62% -0.03%
==========================================
Files 118 118
Lines 13317 13324 +7
==========================================
+ Hits 10740 10742 +2
- Misses 2577 2582 +5
Flags with carried forward coverage won't be shown. Click here to find out more.
|
48d87ac
to
928f327
Compare
added option for explicit rpm cluster CRS with default to WGS 84
No description provided.