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

Fgb async iterator #974

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

H-Plus-Time
Copy link
Contributor

No description provided.

@kylebarron kylebarron changed the base branch from main to kyle/fgb-async-iterator January 13, 2025 16:16
@kylebarron
Copy link
Member

I changed the base to #966 in the hope that it would make this easier to review... not sure I succeeded 😅

@kylebarron kylebarron changed the base branch from kyle/fgb-async-iterator to main January 13, 2025 16:17
@kylebarron
Copy link
Member

Ok switched it back to main 😅

@kylebarron
Copy link
Member

@H-Plus-Time I've been swamped and on vacation; I have a couple days back working before taking Friday and Monday off ⛷️

I really appreciate this PR and hope to get to a review very soon!

@H-Plus-Time
Copy link
Contributor Author

Oh, no rush at all (personally I'd be duct-taping my laptop shut if it were ski season here).

There's a bunch of failing CI, I'll have a bit of a poke around there (I'm guessing it's just combinations of features / slightly outdated imports).

@H-Plus-Time
Copy link
Contributor Author

Mm, ok, I suppose we just restore the read_flatgeobuf_async function (as a simple wrapper around the stream builder struct) and that should silence most of the above.

Interestingly though, I see you've started messing with async iterators over at https://github.com/kylebarron/arro3/pull/258/files. That would be really nice, and I'm guessing that's something you'd want to do in arro3/pyo3_arrow (though I do wonder how it'd work with the dynamically linked bits)?

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.

2 participants