Bioconductor / BiocParallel

Bioconductor facilities for parallel evaluation
https://bioconductor.org/packages/BiocParallel
67 stars 29 forks source link

Force evaluation of the function argument #242

Closed Jiefei-Wang closed 1 year ago

Jiefei-Wang commented 1 year ago

My git skill is a little bit rusty, I pushed to the master branch by accident. I have reverted the change but I think we can do a force reset to clean it up.

This is a pull request to solve the test 3 example in https://github.com/Bioconductor/BiocParallel/issues/241#issuecomment-1444297564. I force R to evaluate all function arguments before serializing the function object. This should solve the issue introduced by the delayed assignment.

Jiefei-Wang commented 1 year ago

Thanks for your comment. I have added a unit test for it. This unit test will fail in the current devel version but will pass in this branch.

mtmorgan commented 1 year ago

merged in b10ee16b98717ab29d1bad73e629f4b08c68d4e6