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

Add kwargs to function generated tables #4542

Merged

Conversation

alexpeters1208
Copy link
Contributor

In working on the function-generated tables blog, I wanted to be able to use arguments in the table-generating function. This PR adds that functionality.

@alexpeters1208 alexpeters1208 self-assigned this Sep 22, 2023
@alexpeters1208 alexpeters1208 changed the title Add kwargs to function generated tables and unit test Add kwargs to function generated tables Sep 22, 2023
@alexpeters1208 alexpeters1208 marked this pull request as ready for review September 22, 2023 21:52
jmao-denver
jmao-denver previously approved these changes Sep 25, 2023
Copy link
Contributor

@jmao-denver jmao-denver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@jmao-denver jmao-denver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor doc issues.

py/server/deephaven/table_factory.py Outdated Show resolved Hide resolved
Co-authored-by: Jianfeng Mao <4297243+jmao-denver@users.noreply.github.com>
Copy link
Contributor

@jmao-denver jmao-denver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alexpeters1208 alexpeters1208 merged commit bd60e3e into deephaven:main Sep 26, 2023
10 checks passed
@alexpeters1208 alexpeters1208 deleted the parameterize-func-generated-table branch September 26, 2023 18:40
@github-actions github-actions bot locked and limited conversation to collaborators Sep 26, 2023
@alexpeters1208 alexpeters1208 linked an issue Sep 27, 2023 that may be closed by this pull request
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pydocs for function_generated_table have weird grey boxes
3 participants