Nick Crews
Nick Crews
> ir.IntegerColumn is an implementation detail Are you sure? `ir.IntegerScalar` does not have a `.max()` method, `ir.IntegerColumn` does. We talked about this [here as well](https://github.com/ibis-project/ibis/pull/8358#discussion_r1489992174). I think if it were...
Thanks! > I can't say I'd really recommend this, because you'd like to keep `ConnectionWrapper ` as non-public, and you might make breaking changes at any time? That seems like...
@Aarya2004 you should be able to create the PR without needing to be an assignee. You will need to make a fork of this repo, push your commits to a...
I case this is useful to others, as a workaround until this is implemented, I wrote a [tiny python library](https://github.com/NickCrews/llama-cpp-server-python) that - downloads and installs the [raw llama.cpp server binary](https://github.com/ggerganov/llama.cpp/tree/master/examples/server)...
Ok, if it's a big burden then I think it's fine to just punt, it's probably not THAT common. Maybe I can fix up this issue title to be SEO...
That looks great. Does the public API I propose of ibis.to_sql() look good to you? Do you know how to map from dialect="duckdb" to the DuckDBType? EDIT: see #11100, I...
@cpcloud this is ready for a review whenever you get the chance! I can talk it through with you to if you want. I'd love to get this in as...
> As an alternative, how about a method on DataType, called to_sqlglot, similar to how Schema.to_sqlglot works? > > Then we can have a very small amount of code in...
Well, it still is sorta ugly, but maybe less ugly than before. Once I added DataType.to_sqlglot() method, then this also begs for a DataType.from_sqlglot(sqlglot_type: sge.DataType) method, for symmetry with to_arrow/from_arrow,...
We already have a similar test that checks `con.execute(ibis.literal("a string").typeof())` at https://github.com/ibis-project/ibis/blob/d55a5eee5a9fa45d53ea51775f53d1215ed819ac/ibis/backends/tests/test_string.py#L78-L117 This is a bit different though. I'm curious though, why that test, paramaterized with the `con` fixture, has...