Phillip Cloud comments

Results 993 comments of


                                            Phillip Cloud

feat: show `df.count(), len(df.schema())` when displaying `interactive = True` tables

> first N/2 rows and last N/2 rows The first and last rows of a table in most of our backends will be completely arbitrary, because a table in a...

feat: show `df.count(), len(df.schema())` when displaying `interactive = True` tables

Not entirely opposed to an option for this but it requires someone else to do the work, and I'll review the PR/help get it over the finish line. A trade-off...

bug: BigQuery backend table.to_pyarrow_batches is not honoring the chunk_size parameter

It's possible that we're not encoding the chunk size in the right way. I recall there being some complexity around how paged results are related to chunks. Maybe @tswast knows:...

bug: BigQuery backend table.to_pyarrow_batches is not honoring the chunk_size parameter

It might be useful to do that, but it seems like it could be counterproductive to incur the overhead of re batching (though I'm not sure how expensive that is).

fix(duckdb): use simple GEOMETRY type for all geospatial data (#10324)

@anjakefala Ping on this! Still interested in adding a test?

fix(duckdb): use simple GEOMETRY type for all geospatial data (#10324)

Your test fails for me on `main` with the same error reported by the user. Are you sure you were running your test against `main`?

fix(duckdb): use simple GEOMETRY type for all geospatial data (#10324)

Thanks!

feat: support `ibis.postgres.connect("postgresql://...")` in addition to `ibis.connect("postgresql://...")`

This doesn't work for any backend. It used to work, and it made the implementation of connect unnecessarily complicated. If you want to use URLs to connect, then you go...

feat: support `ibis.postgres.connect("postgresql://...")` in addition to `ibis.connect("postgresql://...")`

We're already parsing the URL, and the connect-from-URI API and connect-with-args API are well-separated, so I'd rather just keep that as is. +1 to clarifying this in our docs.

bug: type error when accessing `length()` without schema information

I can't reproduce this on `main`: ``` In [4]: import ibis ...: import pandas as pd ...: ...: ...: d = pd.DataFrame( ...: { ...: "some_col": [ ...: [1, 2,...