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.
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
Implement
TripsLayer
for animating moving objects and connect to MovingPandas #292Implement
TripsLayer
for animating moving objects and connect to MovingPandas #292Changes from 1 commit
c582438
1bd1ad3
8075fbb
8e5a479
96ac30b
acaaddb
daae1b4
702a853
159ff0b
d3154b6
0fb89c4
72e0bdb
c707aad
7247452
fa65b87
1ced621
fff32f6
0dcf3fe
a4bc5b9
dd56ef3
9944c66
b32939d
ca9dcd1
37a64c6
6addb2e
b346810
b08965a
d2a5202
150c46b
b94194a
2bbcd3e
ddb64f8
9391062
3297403
6cca53a
28938c8
837bb1d
49b20fc
7d262bf
4b0ee7b
61c5923
e859ce1
5a026eb
3e75895
79cc520
186c419
c790753
b64b330
7f59af9
5247af0
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
This relies on arro3's
list(..)
to convert the pyarrow field through the arrow pycapsule interface, and then pyarrow'swith_type()
convert again in the other direction?Nice ;)
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.
Ah, no, I see that you converted the pyarrow Table to am arro3 one (
attr_table = Table.from_arrow(attr_table)
) before calling this function ;)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.
Here
batch
is actually anarro3.RecordBatch
, the conversion back to arro3 happened here:lonboard/lonboard/_geoarrow/movingpandas_interop.py
Line 116 in 1ced621
But yes, these lines should work with pyarrow input as well!
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.
Right. I could've left it as a pyarrow table; the main difference is mostly for type hinting. It's nice to get IDE completions and type checking and pyarrow doesn't have that yet.