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 clustermq package #10

Merged
merged 1 commit into from
Aug 29, 2017
Merged

add clustermq package #10

merged 1 commit into from
Aug 29, 2017

Conversation

mschubert
Copy link
Contributor

https://github.com/mschubert/clustermq

New on CRAN, difference to available package is extremely simple use (single line), no network-mounted storage and accessing schedulers via SSH.

https://github.com/mschubert/clustermq

New on CRAN, difference to available package is extremely simple use (single line), no network-mounted storage and accessing schedulers via SSH.
@eddelbuettel
Copy link
Owner

Sure, could do. Are you sure you need eight Imports: for your package?

@mschubert
Copy link
Contributor Author

Thanks !

Reducing the number of dependencies has not been a priority so far, I might throw out pryr and infuser at some point (wouldn't really count utils). For purrr, R6 and tibble, I'm pretty sure they are already available in more than 80% of the R installations.

The biggest hassle in terms of deps is rzmq (requires an additional system lib– which is probably worse than all the other pkgs combined), but that's absolutely essential.

@eddelbuettel
Copy link
Owner

For purrr, R6 and tibble, I'm pretty sure they are already available in more than 80% of the R installations.

I beg to differ. Let's just agree to disagree here.

0mq is a different, and a lovely piece of work which Whit kindly wrapped up

@mschubert
Copy link
Contributor Author

Thank you for the feedback. I'll put it on my list for 0.8 to see what I can remove.

@eddelbuettel eddelbuettel merged commit ff7450f into eddelbuettel:master Aug 29, 2017
eddelbuettel added a commit that referenced this pull request Aug 29, 2017
@eddelbuettel
Copy link
Owner

I needed a minor amount of post-processing (date, adding clustermq at bottom, one pass over test) and now also committed at Achim's SVN repo.

Thanks for the PR!

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.

None yet

2 participants