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

Add future_pvec() analogue to parallel::pvec() #96

Open
HenrikBengtsson opened this issue Nov 23, 2021 · 2 comments
Open

Add future_pvec() analogue to parallel::pvec() #96

HenrikBengtsson opened this issue Nov 23, 2021 · 2 comments

Comments

@HenrikBengtsson
Copy link
Collaborator

Not sure how I missed parallel::pvec(). Should add future_pvec() to future.apply.

@HenrikBengtsson HenrikBengtsson added this to the Next release milestone Nov 23, 2021
@HenrikBengtsson
Copy link
Collaborator Author

Oh well, I "missed" it because pvec() is from them parallel package and not part of the base R map-reduce family.

HenrikBengtsson added a commit that referenced this issue Dec 1, 2021
@HenrikBengtsson
Copy link
Collaborator Author

So, I did implement this in commit 6f3bbc2. However, because (i) it's not a base R map-reduce function, and because (ii) parallel::pvec() is rather low-level and does not guarantee consistent results (e.g. still returns if a forked child process crashes), I'll hold back adding future_pvec() to this package.

@HenrikBengtsson HenrikBengtsson removed this from the Next release milestone Dec 1, 2021
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