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

Par framework API cleanup #28

Open
lars-t-hansen opened this issue Jun 8, 2015 · 0 comments
Open

Par framework API cleanup #28

lars-t-hansen opened this issue Jun 8, 2015 · 0 comments

Comments

@lars-t-hansen
Copy link
Owner

Probably the continuation argument should always be the last argument, as that supports a natural JS style:

Par = new MasterPar(..., function () {
  Par.broadcast(..., function () {
    Par.invoke(..., function () {
      ...
    }
  }
}

For that to work out, Par.invoke and Par.broadcast must take the arguments to the workers as an array, not as expanded arguments - probably just as well.

Not sure how important it is. However, if we don't do this then at least move the continuation to be the first argument to the MasterPar constructor, since it is the first argument on the other APIs, including Par.eval.

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

No branches or pull requests

1 participant