You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This will unblock us to fully tackle #223, #227, and #245.
Complications
arrange/distinct introduce one particular issue. We have added the _with prefix to disambiguate the macro-api from the non-macro API. This was easy because the non-macro API for mutate/summarize/filter are function based. However, arrange/distinct already have a non-macro API that is not function based, for example:
arrange(df, desc: "my_field")
But we also want to support this:
arrange(df, desc: my_field)
We have three choices:
Keep arrange(df, desc: "my_field") and arrange(df, desc: my_field), under the same function/arity. This may be doable but it may also raise ambiguities. For example, should we allow arrange(df, desc: my_field, asc: "another-field")?
Move the non-macro API to arrange_with, which will support keywords or functions, such as arrange_with(df, desc: "my_field")
Remove the arrange(df, desc: "my_field") version. People can either use arrange(df, desc: my_field) or arrange_with(df, fn df -> [desc: df["my_field"]] end)
EDIT: distinct has further complications, because the columns are passed as options and we will have to revisit that.
The text was updated successfully, but these errors were encountered:
This adds the basic functionality for the `filter_with/2` function
discussed in #223 and addressed by #289
This change adds two new opaque backends - one for data frame and one for series.
They work in conjunction to create lazy series and accumulate operations on them
before sending to the backend.
The goal is to introduce
filter_with
,summarize_with
,mutate_with
, andarrange_with
.Attack plan
filter_with
with row-based series operationssummarize_with
with aggregation-based series operationsmutate_with
with row, group, and aggregation-based series operationsarrange_with
This will unblock us to fully tackle #223, #227, and #245.
Complications
arrange
/distinct
introduce one particular issue. We have added the_with
prefix to disambiguate the macro-api from the non-macro API. This was easy because the non-macro API formutate
/summarize
/filter
are function based. However,arrange
/distinct
already have a non-macro API that is not function based, for example:But we also want to support this:
We have three choices:
Keep
arrange(df, desc: "my_field")
andarrange(df, desc: my_field)
, under the same function/arity. This may be doable but it may also raise ambiguities. For example, should we allowarrange(df, desc: my_field, asc: "another-field")
?Move the non-macro API to
arrange_with
, which will support keywords or functions, such asarrange_with(df, desc: "my_field")
Remove the
arrange(df, desc: "my_field")
version. People can either usearrange(df, desc: my_field)
orarrange_with(df, fn df -> [desc: df["my_field"]] end)
EDIT:
distinct
has further complications, because the columns are passed as options and we will have to revisit that.The text was updated successfully, but these errors were encountered: