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

Fix df/series.{apply, map_chunk} when some chunk output empty data #2434

Merged
merged 2 commits into from
Sep 7, 2021

Conversation

chaokunyang
Copy link
Contributor

What do these changes do?

Fix empty series/dataframe apply error.

Related issue number

closes #2433

mars/dataframe/base/apply.py Outdated Show resolved Hide resolved
@qinxuye qinxuye changed the title fix empty df/series apply Fix df/series.{apply, map_chunk} when some chunk output empty data Sep 7, 2021
@qinxuye qinxuye added mod: dataframe to be backported Indicate that the PR need to be backported to stable branch type: bug Something isn't working labels Sep 7, 2021
@qinxuye qinxuye added this to In progress in DataFrame via automation Sep 7, 2021
@qinxuye qinxuye added this to PR-In progress in v0.8 Release via automation Sep 7, 2021
@qinxuye qinxuye added this to the v0.8.0b1 milestone Sep 7, 2021
Copy link
Collaborator

@qinxuye qinxuye 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
Member

@wjsi wjsi left a comment

Choose a reason for hiding this comment

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

LGTM

DataFrame automation moved this from In progress to Reviewer approved Sep 7, 2021
@wjsi wjsi merged commit 7221874 into mars-project:master Sep 7, 2021
DataFrame automation moved this from Reviewer approved to Done Sep 7, 2021
v0.8 Release automation moved this from PR-In progress to PR-Done Sep 7, 2021
qinxuye pushed a commit to qinxuye/mars that referenced this pull request Sep 8, 2021
wjsi pushed a commit that referenced this pull request Sep 8, 2021
@wjsi wjsi added backported already PR has been backported and removed to be backported Indicate that the PR need to be backported to stable branch labels Sep 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backported already PR has been backported mod: dataframe type: bug Something isn't working
Projects
DataFrame
  
Done
Development

Successfully merging this pull request may close these issues.

[BUG] KeyError was raised when execute dataframe getitem
3 participants