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

Provide experimental API for changing number of return values and res… #1735

Merged
merged 2 commits into from
Mar 19, 2018
Merged

Provide experimental API for changing number of return values and res… #1735

merged 2 commits into from
Mar 19, 2018

Conversation

robertnishihara
Copy link
Collaborator

@robertnishihara robertnishihara commented Mar 18, 2018

…ource requirements at task submission time.

@devin-petersohn can you try this out and see if it does what you want? You can use it with

import ray

ray.init()

@ray.remote
def f(n):
    return n * [1]

a, b, c = f._submit(args=[3], num_return_vals=3)

TODO before merging:

  • Add tests.
  • Remove code duplication between ._submit and .remote.
  • Make sure this doesn't mess up the error messages.
  • Make sure it doesn't affect performance.

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-PRB/4380/
Test FAILed.

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-PRB/4389/
Test PASSed.

@pcmoritz pcmoritz self-requested a review March 19, 2018 22:32
@pcmoritz pcmoritz merged commit d78de0d into ray-project:master Mar 19, 2018
@pcmoritz pcmoritz deleted the numreturnvalues branch March 19, 2018 22:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants