-
Notifications
You must be signed in to change notification settings - Fork 132
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
GeoJSON functions #301
Merged
GeoJSON functions #301
Changes from 76 commits
Commits
Show all changes
92 commits
Select commit
Hold shift + click to select a range
39321be
polygons_to_cells
ajfriend 85d112e
Helpers for cells_to_geojson and geojson_to_cells
ajfriend cfc6ba3
ValueError
ajfriend 0ae6115
Remove gj (#316)
ajfriend d41eb15
actions/[email protected]
ajfriend cad94ca
Solution copied from https://github.com/actions/setup-python/issues/544
ajfriend 66c2731
Merge branch 'master' into poly
ajfriend 299b756
playing around with polygons
ajfriend b997c73
make a directory
ajfriend 93fcfef
bah
ajfriend 49f61c0
start migrating `h3.Polygon` to `H3Poly`
ajfriend 3b641a8
migrate most
ajfriend a85544a
finish migrating
ajfriend 91560a8
H3MultiPoly repr
ajfriend 1d4da65
shape_to_cells and cells_to_shape
ajfriend 6786132
tests: polygon_to_cells to shape_to_cells
ajfriend c0cdc50
finish migrating tests
ajfriend 64476c7
update util.py and example
ajfriend 5e06441
notes
ajfriend c436bda
bah
ajfriend 5655624
fix lint
isaacbrodsky 5d6ccef
fix
isaacbrodsky 4cb001d
remove X_to_shape
isaacbrodsky e0b5afd
run tests
isaacbrodsky ffd8e54
run other CIs too
isaacbrodsky 91fa567
don't use f-string
isaacbrodsky 02e912b
test update
isaacbrodsky 33660ae
change errors per review
isaacbrodsky 1eebc70
Merge branch 'master' into poly-2
isaacbrodsky 178065f
remove unused per review
isaacbrodsky f681b0b
fix lint
isaacbrodsky 5c7614e
Merge pull request #322 from isaacbrodsky/poly-2
isaacbrodsky f6f8b58
test run
ajfriend 18e0c10
fix exclude logic in tests
ajfriend 311dd88
" to ' in yaml
ajfriend 6f107a1
move conversion logic to geo_to_cells function
ajfriend 22cab2e
linting
ajfriend b31a6fc
few more tests
ajfriend f63c0da
lint
ajfriend 96b17ce
clean up
ajfriend f0ef783
complete tests
ajfriend f27f1b4
group geojson/polygon tests into a folder
ajfriend 7e8f20c
move MockGeoInterface into the file that uses it
ajfriend ba29eb8
move test_input_format to its own file, since it has a lot of helper …
ajfriend 82a23bc
test module description
ajfriend aad0a08
rename _polygon.py to _h3shape.py
ajfriend 5680314
make H3Shape an ABC
ajfriend 8ab1cc1
shape -> h3shape
ajfriend ae33ac5
geos to polys
ajfriend df70c05
rename geo files to latlng
ajfriend 35b7ace
docstrings
ajfriend f320c13
__init__.py
ajfriend 56c35d8
lint
ajfriend ffa451b
notebooks
ajfriend 75640d0
more interface tests
ajfriend dbfa39d
notebooks
ajfriend 23dfc3e
merp
ajfriend 3cac289
clean up geo_to_h3shape()
ajfriend 3810cd3
simplify sydney coords
ajfriend 72e6643
simplify far east test
ajfriend 11befdc
simplify sf coords
ajfriend 6725d98
tuples
ajfriend 8922cc8
geo interface tests
ajfriend e9041fc
more interface tests
ajfriend 08e1201
clean up maine lat/lngs
ajfriend 5f37f43
messed up lat/lng order in tests. this mistake gets made all the time!
ajfriend 1fea86c
expand geo_to_h3shape test
ajfriend d52679a
plotting with contextily and geoviews
ajfriend d60a5af
reprs for H3Shape objects should work better with geopandas
ajfriend 297ecf6
different representations of the same information. which is best?
ajfriend 07850dc
us "loopcode" in H3Poly and H3MultiPoly reprs
ajfriend acd629d
set tight=True as default
ajfriend 86d7f2a
can't use f-strings in python 3.5
ajfriend 58ea719
notebooks
ajfriend 026f96e
basic plotting
ajfriend 3a0c5af
plotting
ajfriend 1210106
plotting
ajfriend 78b5a6d
tutorial notebook
ajfriend 31e3fcf
cleaning up
ajfriend ce361d6
clean up geoviews example
ajfriend d8ea193
remove geoviews example
ajfriend e822e10
todos
ajfriend 461ef90
Merge branch 'master' into test_poly
ajfriend 88f9ab4
reworking tutorial order
ajfriend cab31d8
notebook dump
ajfriend 76f9965
cleaner
ajfriend 7001e71
move tutorial
ajfriend 2b4b249
Update src/h3/api/_api_template.py
ajfriend ef1ac95
Update src/h3/api/_api_template.py
ajfriend 6b46459
Update src/h3/api/_api_template.py
ajfriend bee3bd2
setting loops
ajfriend dc2f550
bug and lint
ajfriend File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,7 +4,7 @@ on: | |
push: | ||
branches: [master] | ||
pull_request: | ||
branches: [master] | ||
branches: ['*'] | ||
|
||
jobs: | ||
tests: | ||
|
@@ -18,7 +18,7 @@ jobs: | |
|
||
- uses: actions/[email protected] | ||
with: | ||
python-version: 3.8 | ||
python-version: 3.11 | ||
|
||
- name: Install from source | ||
run: | | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,7 +4,7 @@ on: | |
push: | ||
branches: [master] | ||
pull_request: | ||
branches: [master] | ||
branches: ['*'] | ||
|
||
jobs: | ||
tests: | ||
|
@@ -15,6 +15,7 @@ jobs: | |
matrix: | ||
os: [macos-latest, windows-latest, ubuntu-20.04] | ||
python-version: ['3.5', '3.6', '3.7', '3.8', '3.9', '3.10', '3.11'] | ||
exclude: | ||
- os: windows-latest | ||
python-version: 3.5 | ||
|
||
|
@@ -25,7 +26,7 @@ jobs: | |
|
||
- uses: actions/[email protected] | ||
with: | ||
python-version: "${{ matrix.python-version }}" | ||
python-version: '${{ matrix.python-version }}' | ||
|
||
## Start Windows stuff | ||
- uses: ilammy/[email protected] | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Were the quotes changed by a yaml formatter? If not I would recommend having this enforced by formatted rather than manual changes