Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] fexpr unary operators #3449

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Conversation

samukweku
Copy link
Contributor

@samukweku samukweku self-assigned this Mar 30, 2023
@samukweku
Copy link
Contributor Author

@oleksiyskononenko i'm stuck on how to adapt the code for unary minus and sorting, i.e dt.sort(-f.A) or dt.by(-f.A)

@oleksiyskononenko
Copy link
Contributor

@samukweku You need to rebase this PR on top of main, otherwise it is showing too many old changes.

@samukweku samukweku force-pushed the samukweku/fexpr_unary_operators branch from 2fad60d to d4f9a91 Compare March 31, 2023 01:55
@samukweku samukweku force-pushed the samukweku/fexpr_unary_operators branch from d27315b to 8a78a14 Compare April 11, 2023 10:50
@samukweku
Copy link
Contributor Author

samukweku commented Apr 15, 2023

@oleksiyskononenko this is red because I do not know how to adapt the code for unary minus and sorting, i.e dt.sort(-f.A) or dt.by(-f.A). I would appreciate your guidance on this one

@samukweku samukweku force-pushed the samukweku/fexpr_unary_operators branch from 3e08c55 to 4109eb6 Compare April 20, 2023 23:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants