-
Notifications
You must be signed in to change notification settings - Fork 325
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
Support fetch executed tensor data from server #80
Conversation
…em&getitem # Conflicts: # mars/web/tests/test_api.py
…_tensor # Conflicts: # mars/deploy/local/tests/test_cluster.py # mars/web/session.py
Wonder why local session cannot fetch? |
okay... |
Yeah, add an interface to Executor to supply |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
What do these changes do?
Add
session.fetch
method and check if tensor has been submitted insession.run
. ForLocalClusterSession
andweb.Session
, they can fetch executed tensor from workers and submit other tensors to cluster. ForLocalSession
, fetch is not available, it will run all input tensors whether they have run or not.Related issue number
fixes #17 from the client side, other related fixes may be done with #28