Maximilian Roos
Maximilian Roos
> I did add some nans (`x[x < 0.0] = np.nan`) and it works. Great! OK that is quite encouraging...
Awesome! Nice find. Definitely interested in a PR — would be keen to see how the tests do. Thanks! (FYI am traveling in India so may be a bit slow...
Hi @andygrove ! Is the proposal to bring the code into a data fusion _crate_? Or just the data fusion repo?
Ah cool! No view in that case. (From a very happy & grateful user, thanks for the library!)
Thanks for the swift response @alamb > 👋 How about https://docs.rs/sqlparser/0.33.0/sqlparser/ast/enum.Expr.html#variant.BinaryOp ? The only one there which is custom is [`PGCustomBinaryOperator`](https://docs.rs/sqlparser/0.33.0/sqlparser/ast/enum.BinaryOperator.html#variant.PGCustomBinaryOperator). The official [docs](https://www.postgresql.org/docs/current/sql-createoperator.html) of that don't allow for words...
> I think https://github.com/sqlparser-rs/sqlparser-rs#syntax-vs-semantics may be relevant here OK — I had thought that allowed characters might be syntax rather than semantics, but very happy to take it! Thanks @alamb...
Sorry, actually I spoke too soon — that seems to [write `OPERATOR` when writing it out](https://github.com/sqlparser-rs/sqlparser-rs/blob/1cf913e7173a27897c0d198782e385de806dbcbd/src/ast/operator.rs#L136). This would be the diff I need — happy to put a PR in...
> If so, why not add a regex operator and then for the MySQL/sqlite dialects parse it as a binary op? Initial effort at https://github.com/sqlparser-rs/sqlparser-rs/pull/874 !
I've been using [watchexec](https://github.com/watchexec/watchexec) as a more general watcher, since pytest-watch no longer works for me. (This was before I saw @olzhasar 's https://github.com/olzhasar/pytest-watcher, which looks great for python-only use...
IIUC, it's worth checking out https://github.com/aj-bagwell/clio, which does some of these...