8000 Add dataframe.join by MarcoGorelli · Pull Request #238 · data-apis/dataframe-api · GitHub
[go: up one dir, main page]
Skip to content
8000

Add dataframe.join #238

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.

Sign up for GitHub

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

Merged
merged 2 commits into from
Aug 28, 2023
Merged

Add dataframe.join #238

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
a5f5954
add dataframe.join
MarcoGorelli Aug 26, 2023
2978c33
remove on
MarcoGorelli Aug 28, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
8000
Diff view
42 changes: 42 additions & 0 deletions spec/API_specification/dataframe_api/dataframe_object.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -897,3 +897,45 @@ def to_array_object(self, dtype: Any) -> Any:
understanding that consuming libraries would then use the
``array-api-compat`` package to convert it to a Standard-compliant array.
"""

def join(
self,
other: DataFrame,
*,
how: Literal['left', 'inner', 'outer'],
on: str | list[str] | None = None,
left_on: str | list[str] | None = None,
right_on: str | list[str] | None = None,
) -> DataFrame:
"""
Join with other dataframe.

Parameters
----------
other : DataFrame
Dataframe to join with.
how : str
Kind of join to perform.
Must be one of {'left', 'inner', 'outer'}.
Comment on lines +916 to +918
Copy link
Collaborator

Choose a reason for hiding this comment

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

I know right joins can be modeled as left joins with the inputs swapped, but any reason not to support right as a join type here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I know right joins can be modeled as left joins with the inputs swapped

exactly, so if the idea is to make a minimal API, right joins probably aren't needed. have never seem them in practice anyway

Copy link
Collaborator

Choose a reason for hiding this comment

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

We can always add them later without it being a breaking change, so I'm good with not including them until there's clear feedback that they're needed / wanted.

on : str | list[str], optional
Key(s) to use to perform join.
Cannot be used in conjunction with `left_on` or `right_on`.
left_on : str | list[str], optional
Key(s) from `self` to perform `join` on.
If more than one key is given, it must be
the same length as `right_on`.
Cannot be used in conjunction with `on`.
right_on : str | list[str], optional
Key(s) from `other` to perform `join` on.
If more than one key is given, it must be
the same length as `left_on`.
Cannot be used in conjunction with `on`.

Returns
-------
DataFrame

Notes
-----
Either `on`, or both `left_on` and `right_on`, must be specified.
"""
0