Hi Michael, 2018-03-12 8:44 GMT+01:00 Michael Cree <[email protected]>: > How about the attached patch? Should detection of number of cores > fail it sets the number of cores to be one. With this patch > r-cran-nmf builds successfully on Alpha.
Thanks for this patch. But, I think if we really want to fix this issue in our package before than upstream releases a fixed version, we should probably use the same way as upstream. That is to backport these 2 commits [1-2]. Best, Dylan [1] https://github.com/renozao/NMF/commit/063cdcb102 [2] https://github.com/renozao/NMF/commit/6996bce01e
