Use of requireNamespace() not really needed - can be dropped

[HenrikBengtsson]

Hello, just a drive-by comment. I see you're using requireNamespace() in:

interflex/R/kernel.R

Lines 763 to 768 in e35ba4a

	requireNamespace("doParallel")
	## require(iterators)
	maxcores <- detectCores()
	cores <- min(maxcores, cores)
	pcl <- future::makeClusterPSOCK(cores)
	doParallel::registerDoParallel(pcl)

Note that it makes no difference if you have it or not. If the doParallel package is not installed, then requireNamespace("doParallel") will silently return FALSE and continue. The real test is in doParallel::registerDoParallel(pcl), which will give an error if doParallel is not installed.

I suggest dropping requireNamespace() in these cases, because it has no effect and makes you wonder what the purpose is.

[xuyiqing]

Thank you. We will fix it in the next update with doFuture Sent from IPhone.

…

On Sun, Nov 16, 2025 at 12:23 PM Henrik Bengtsson ***@***.***> wrote: *HenrikBengtsson* created an issue (xuyiqing/interflex#21) <#21> Hello, just a drive-by comment. I see you're using requireNamespace() in: https://github.com/xuyiqing/interflex/blob/e35ba4a414721da1874caa6759cc33055494fa98/R/kernel.R#L763-L768 Note that it makes no difference if you have it or not. If the *doParallel* package is *not* installed, then requireNamespace("doParallel") will silently return FALSE and continue. The real test is in doParallel::registerDoParallel(pcl), which will give an error if *doParallel* is not installed. I suggest dropping requireNamespace() in these cases, because it has no effect and makes you wonder what the purpose is. — Reply to this email directly, view it on GitHub <#21>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AB2PKGCIOVBW4SVUQ3XILG335DMLVAVCNFSM6AAAAACMIUIJXSVHI2DSMVQWIX3LMV43ASLTON2WKOZTGYZTCMBZGI2DQNI> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>