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

Set of small features #839

Merged
merged 11 commits into from
Sep 2, 2024
Merged

Conversation

timsaucer
Copy link
Contributor

Which issue does this PR close?

Closes #713
Closes #807
Closes #810

Also no issue assigned but fixes a bug in functions.array()

Rationale for this change

These are a handful of usability improvements.

What changes are included in this PR?

  • Adds _repr_html_ to get a nice output in notebooks
  • Allows for using [] to access array elements in an expression. If column a is a list of integers you could do col(a)[0] to get the first element in the list.
  • Fixes an error in array so it operates correctly.
  • Adds DataFrame.transform() which makes chaining dataframe operations easier.
reprhtml

Are there any user-facing changes?

No API changes, but the above all enhance usability.

@timsaucer
Copy link
Contributor Author

timsaucer commented Aug 25, 2024

TODO:

  • Add unit tests for dataframe transform
  • Add unit test for repr_html
  • Add documentation example for chaining dataframe operations
  • Add documentation for how to access array and struct elements to the user guide

@timsaucer timsaucer marked this pull request as ready for review August 27, 2024 12:01
@timsaucer timsaucer mentioned this pull request Aug 27, 2024
9 tasks
@timsaucer
Copy link
Contributor Author

@andygrove These should be good to go. All of the conflicts were trivial "include both" on the rebase.

@andygrove andygrove merged commit 909b809 into apache:main Sep 2, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants