-
Notifications
You must be signed in to change notification settings - Fork 0
Description
Hi! Thanks for linking here [from this duckdb-python](duckdb/duckdb-python#370 (comment) discussion)! This is a cool project, nice job so far!
two things:
Have you seen ibis? It would be useful to put in your readme how this is different from ibis. Or, we at ibis would love to hear why ibis isn't good enough for your needs, so that we can improve. Finally, the biggest stretch: If we could convince you that ibis was a better investment of your energy, we would love to have you contribute there and unify our efforts!
second thing:
The real missing feature that in MY opinion ibis is terrible at is typing. If you do keep going with this, the reason that I might switch to a different dataframe library is for excellent typing support. For example, tracking the schema of individual columns as a relations goes through the transofrmations. This isn't currently possible with python's type system, but https://peps.python.org/pep-0827/ tries to fix that. Ibis is way to big and mature at this point for me to really want to take on migrating to this typing feature, but you might havehope here! So I guess I'm just suggesting you keep that PEP in mind, maybe experiment with it here, and keep your architecture of this lib open for adding this typing support in the future if that PEP is accepted.