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

Allow passing in a string transform for tbl.update_spec().add_field() #1882

Open
Fokko opened this issue Apr 4, 2025 · 0 comments
Open

Allow passing in a string transform for tbl.update_spec().add_field() #1882

Fokko opened this issue Apr 4, 2025 · 0 comments
Labels
good first issue Good for newcomers

Comments

@Fokko
Copy link
Contributor

Fokko commented Apr 4, 2025

Feature Request / Improvement

Today you have to pass in an instance of the transform:

with  tbl.update_spec() as upd:
    upd.add_field("tpep_pickup_datetime", DayTransform())

Instead, I think it would be great to also pass in the name:

with  tbl.update_spec() as upd:
    upd.add_field("tpep_pickup_datetime", "day")

This should be pretty straightforward using the parse_transform function.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant