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 recursive_tile that it may cause duplicated tile for one tileable #3021

Merged
merged 1 commit into from
May 12, 2022

Conversation

qinxuye
Copy link
Collaborator

@qinxuye qinxuye commented May 11, 2022

What do these changes do?

This PR fixed recursive_tile that it may cause duplicated tile for one tileable.

Related issue number

Fixes #3010

Check code requirements

  • tests added / passed (if needed)
  • Ensure all linting tests pass, see here for how to run them

@qinxuye qinxuye added type: bug Something isn't working to be backported Indicate that the PR need to be backported to stable branch mod: task service labels May 11, 2022
@qinxuye qinxuye added this to the v0.10.0a1 milestone May 11, 2022
@qinxuye qinxuye requested review from wjsi and hekaisheng as code owners May 11, 2022 10:37
@qinxuye qinxuye force-pushed the bugfix/recursive_tile branch from a8efee7 to 4bc47ce Compare May 11, 2022 10:57
Copy link
Contributor

@hekaisheng hekaisheng 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

@wjsi wjsi merged commit 44ed47b into mars-project:master May 12, 2022
@qinxuye qinxuye deleted the bugfix/recursive_tile branch May 12, 2022 05:53
qinxuye pushed a commit to qinxuye/mars that referenced this pull request May 23, 2022
wjsi pushed a commit that referenced this pull request May 23, 2022
@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 May 23, 2022
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: task service type: bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Duplicate operands execution
3 participants