Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

Use progressr with future framework for progress updates #703

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ycl6
Copy link

@ycl6 ycl6 commented Sep 28, 2023

This PR changes how some functions deal with sequential and parallel processes.

Following the suggestion in #424 and discussion in #140 regarding the missing future.seed setting, I dropped the detection of number of cores and the ifelse switch between pbapply::pbsapply and future.apply::future_sapply for sequential and parallel computing respectively, and opted to use the progressr package for progress updates. The package came installed with CellChat, so there is no need to install additional dependencies.

I also:

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant