mlr-org / parallelMap

R package to interface some popular parallelization backends with a unified interface
https://parallelmap.mlr-org.com
Other
57 stars 14 forks source link

Implement batchtools #40

Closed jakob-r closed 8 years ago

berndbischl commented 8 years ago

I cannot merge this until bt is on CRAN? Or can you add a mechinism so we can include packages which are only on github?

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-0.4%) to 83.724% when pulling 82f077216bbb9f7c75aa82c78df6500f3a9aa2b6 on jakob-r:feature_batchtools into 4c3754665fed7d2b81bdcba77305e1cdc4a2f52b on berndbischl:master.

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-0.4%) to 83.724% when pulling 936796c598289225c6dc8c16640bd5a523914490 on jakob-r:feature_batchtools into 4c3754665fed7d2b81bdcba77305e1cdc4a2f52b on berndbischl:master.

jakob-r commented 8 years ago

I don't fully understand why appveyor failed. It exceeded one hour? For this one test run?

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-0.2%) to 83.918% when pulling 21b0e9fb0e6fdcb6d862da000aca8197f435b18e on jakob-r:feature_batchtools into 4c3754665fed7d2b81bdcba77305e1cdc4a2f52b on berndbischl:master.

jakob-r commented 8 years ago

First there was a problem because RSQLite was too old so i wanted to take the chance to switch to container structure but now it can not find mpi.h although it OpenMPI seems to be successfully installed and exported. Need help.

mllg commented 8 years ago

Quick solution: Don't do containers but install apt packages (see batchtools' .travis.yml):

sudo: required
dist: trusty
language: r

r:
  - oldrel
  - release
  - devel

addons:
  apt:
    packages:
      - r-cran-rmpi
      - libssh2-1-dev

after_success:
  - Rscript -e 'library(covr);coveralls()'
mllg commented 8 years ago

Merged into branch

jakob-r commented 8 years ago

You mean you work on it here: https://github.com/berndbischl/parallelMap/tree/jakob-r-feature_batchtools So I will close it here to avoid confusion