-
Notifications
You must be signed in to change notification settings - Fork 63
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
check_future_plan(): Variable worker
not always defined
#516
Comments
FYI, you can reproduce this by emulating running on a single-core machine when checking, e.g. R_PARALLELLY_AVAILABLECORES_SYSTEM=1 R CMD check greta_0.4.2.tar.gz This will cause |
Hi @HenrikBengtsson sorry for not replying to this sooner!
This sounds like not a terrible result, if it ends up being sequential - I'm just trying to wrap my head around whether I should try and change the test, or change something in Lines 552 to 559 in 4f61e0b
|
If if (inherits(workers, "cluster")) {
worker <- workers[[1]]
if (!is.null(worker$host)) {
localhosts <- c("localhost", "127.0.0.1", Sys.info()[["nodename"]])
plan_is$local <- worker$host %in% localhosts
}
} might work. Though, I would start by making sure you can reproduce my original error, and then make sure it works after the fix. |
Hi @HenrikBengtsson - sorry for the delay on my end, I'm going to try and tackle this and #513 this week. |
No worries and no rush |
OK, so I get the same error:
As well as several other related errors, going to change the code as you've suggested above :) |
It seems that resolving this in #536 would also resolve #513, does that sound right to you, @HenrikBengtsson ? |
I think they're different. Fix one at the time. This one is trivial to fix, the other one probably needs some troubleshooting. But fix this one first. |
Running revdep checks on parallelly and future, I got:
This is because
worker
is not guaranteed to be defined in:greta/R/checkers.R
Lines 552 to 559 in 4f61e0b
The text was updated successfully, but these errors were encountered: