Comments (2)
Hi - thanks for opening this. Two things:
-
Our type annotations aren't great. Ibis code is pretty dynamic, generally you can trust the output of the method/function/whatever to be properly typed, but the types of arguments are likely stricter than things that are actually supported. In this case here
filter
is annotated to take aBooleanValue
(which will accept aBooleanScalar
orBooleanColumn
), but it also will work with a standardbool
(ibis
will automatically coerce to aBooleanScalar
). Going through the APIs to improve the accuracy of the type annotations and add tests to ensure they're accurate is a large project and not one we're prioritizing at the moment. Right now I recommend either submitting PRs to fix annotations for bugs that you run into, or (better, more feasible suggestion) just turn off typing foribis
. -
In your specific case this is also a case of user error. The
.equals
method on an expression checks for expression-level equality (whether the left and right side are identical expressions).t.filter(t.a == 0)
is what you want here.
from ibis.
Thanks for the feedback - it all makes sense. I'm happy to close this as this is not really a bug, and I see if I can come up with a PR that addresses the specific issue here.
from ibis.
Related Issues (20)
- feat(pyspark): support udaf
- bug HOT 1
- bug: Oracle Table alias HOT 8
- bug: Installation issue: mamba on Windows 10 HOT 6
- feat: Table.to_records()
- feat: `backend.db_params`
- `collect` does not respect order HOT 2
- bug Error when connecting to Trino after upgrading to 9.0.0
- bug: table formatting characters don't render monospace with some fonts HOT 5
- bug: `read_parquet` and similar methods silently overwrite tables HOT 2
- bug: names_sort argument in table.pivot_wider has no effect HOT 1
- feat: add a method for table existence check HOT 5
- bug: `create_table(temp=True)` timing out due to slow table existence check HOT 2
- add support for TIMESTAMPTZ HOT 4
- feat: add a table_exists(table_name) api HOT 1
- bug: `to_sql` always shows DuckDB SQL for a memtable even if there's a default backend set HOT 1
- Polars backend can read only 1 csv HOT 1
- docs: add ops docstrings
- ci: more granular cloud run labels
- feat: allow to connect to a duckdb named in-memory database
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from ibis.