Nick Crews
Nick Crews
Closing as stale. I'll need some more active help for this to happen.
Assuming that CI passes, this is ready for review!
Im going to wait on working on this until more of my easier PRs get merged
In [`92922e3` (#10518)](https://github.com/ibis-project/ibis/pull/10518/commits/92922e3c37c8c4d23674c505d48f0510738bd6c9) I added an optional feature I can remove if you don't like it: the first time you call to_rich_table(), if the show_count is False, we append a...
@cpcloud if I get a +1 to this behavior and implementation, I can do the grunt work of transitioning all the docstrings. If you know of a way to automate...
Ah, I think the actual bug is that the order_by clause isn't properly transferred to the window Op from the First op. If you do `t.mutate(x.first().over(order_by="y", group_by="x"))` then it works.
Also there is a separate bug where we should be forwarding the `RESPECT NULLS`/`IGNORE NULLS to sqlglot depending on if our Operation has `ignore_null` of True or False.
marking as WIP until #11314 is resolved, because that is a better implementation.
My current understanding is that the order_by isn't hoisted from the ops.First() into the ops.WindowFunction(). The best fix for this I have is https://github.com/ibis-project/ibis/pull/11314. The other PRs linked above are...
OK I just did a quick prototype. Per some googling, I found [this article,](https://www.lucidchart.com/techblog/2019/12/06/json-compression-alternative-binary-formats-and-compression-methods/) which recommends brotli for JSON. Brotli is well-defined, mature, and available in most languages. Here is...