The Wayback Machine - https://web.archive.org/web/20201016061819/https://github.com/ray-project/ray/issues/11018
Skip to content

/ ray

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

Jump to bottom

[Placement Group] Not working with @ray.remote #11018

Open
rkooo567 opened this issue Sep 24, 2020 · 2 comments
Open

[Placement Group] Not working with @ray.remote #11018

rkooo567 opened this issue Sep 24, 2020 · 2 comments
Assignees
@clay4444
Labels
P2 bug good first issue
Milestone
Placement Group API

Comments

@rkooo567
Copy link
Contributor

@rkooo567 rkooo567 commented Sep 24, 2020

What is the problem?

Although @ray.remote contains an argument for placement group, it doesn't work. We should either remove it or allow it. (placement_group_capture_child_task)

Reproduction (REQUIRED)

Please provide a script that can be run to reproduce the issue. The script should have no external library dependencies (i.e., use fake or mock data / environments):

If we cannot run your script, we cannot fix your issue.

  • I have verified my script runs in a clean environment and reproduces the issue.
  • I have verified the issue also occurs with the latest wheels.
@rkooo567 rkooo567 added this to the Placement Group API milestone Sep 24, 2020
@rkooo567 rkooo567 added P1 good first issue P2 and removed triage P1 labels Sep 24, 2020
@rkooo567
Copy link
Contributor Author

@rkooo567 rkooo567 commented Oct 8, 2020

I think we shouldn't allow adding placement group in @ray.remote. What do you think @ffbin?
@ffbin
Copy link
Contributor

@ffbin ffbin commented Oct 8, 2020

Yeah, I think so, too.
@ffbin ffbin assigned clay4444 and unassigned ffbin Oct 10, 2020
@clay4444 clay4444 mentioned this issue Oct 14, 2020
Open
0 of 6 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 bug good first issue
3 participants
@ffbin @clay4444 @rkooo567
You can’t perform that action at this time.