Nick Crews comments

Results 452 comments of


                                            Nick Crews

feat: assert_tables_equal()

ahh, that is for abstract structural equality. I am looking for the actual bound data to be the same. Updated my original post to be more clear.

bug: Table.get_name() errors on any non-physical tables

Ok, that's fine with me if we don't support naming tables. Any thoughts on - should we autogenerate a name for all tables? Or - should we move the get_name()...

bug: Table.get_name() errors on any non-physical tables

That seems like a good idea for now. Can add a workaround back in later.

fix: make column.unnest().topk() work

I split out the test refactoring into #11315, maybe we merge that and then I revade this to only include the behavior changes

fix: make column.unnest().topk() work

@cpcloud This is now a lot simpler now that the test refactor is merged separately, and hopefully should be a quick review.

feat: add **options to pyspark backend create_table

@mark-druffel @RohanTemgire please comment on that linked PR, neither me nor Phillip know much about pyspark

[Python] Nullability of Arrow table fields are not respected when table is registered with DuckDB.

I think this is related to https://github.com/duckdb/duckdb/issues/13501?

feat: Support `type` arg to `ibis.array` and `ibis.map`

That would work. Also, that PR is turning into a bit of a doozy, if you wanted to help out there it would be appreciated!

feat: Support `type` arg to `ibis.array` and `ibis.map`

#9473 needs to be resolved for the array changes (which really is getting stalled by larger philosophical decisions that this PR is exposing), and then the linked map PR can...

Is this project dead?

I've been looking around for alternatives, haven't found any yet. I'm currently experimenting with training a custom spaCy NER model using the training data from this repo. You can see...