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

feat(api): add catalog and database kwargs to ibis.table #8801

Merged
merged 1 commit into from
Mar 28, 2024

Conversation

gforsyth
Copy link
Member

This allows specifying 1 or 2 levels of table positional hierarchy in a
call to ibis.table. I've updated the reprs for UnboundTable and
DatabaseTable to show that hierarchy when it is specified.

I've also swapped the attribute order for ops.Namespace as a
convenience to allow unpacking the argnames in the correct hierarchical
order.

Resolves #8253

@gforsyth gforsyth force-pushed the unbound_namespace branch 2 times, most recently from 43a9f16 to e89ced0 Compare March 27, 2024 20:28
@cpcloud cpcloud added this to the 9.0 milestone Mar 27, 2024
@cpcloud
Copy link
Member

cpcloud commented Mar 27, 2024

Looks like rogue parenthesis decided it didn't need to exist!

This allows specifying 1 or 2 levels of table positional hierarchy in a
call to `ibis.table`. I've updated the reprs for `UnboundTable` and
`DatabaseTable` to show that hierarchy when it is specified.

I've also swapped the attribute order for `ops.Namespace` as a
convenience to allow unpacking the argnames in the correct hierarchical
order.
@gforsyth gforsyth force-pushed the unbound_namespace branch from e89ced0 to bcf332d Compare March 27, 2024 21:10
@cpcloud cpcloud added feature Features or general enhancements ux User experience related issues labels Mar 28, 2024
@cpcloud
Copy link
Member

cpcloud commented Mar 28, 2024

👌🏻 Nice UX improvement, thanks!

@cpcloud cpcloud merged commit 7d593c4 into ibis-project:main Mar 28, 2024
85 checks passed
@gforsyth gforsyth deleted the unbound_namespace branch March 28, 2024 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Features or general enhancements ux User experience related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat(table): add namespace kwarg to ibis.table
2 participants