a warning during R CMD INSTALL cranvas about cranvas HOT 7 CLOSED

Just look at the qtour.R file, dest is a field defined in a reference class, need to use <<- to change it, so the warning message can be fixed by change the line 65 dest[[col]] <- data_proj[, col] to dest[[col]] <<- data_proj[, col]

from cranvas.

Weird, my comment didn't add. But I don't think that fix is correct. I think that error is a spurious warning and we should just ignore for now.

from cranvas.

That fix works for me when install cranvas, all other fields in Tourr ref class is assigned by <<-, I mean I always came across some similar warnings like that when I forget to use a "<<- " on the field variable. So that fix is just my guess on that.

from cranvas.

Yes, it won't make an error any more, but does the code still work?

from cranvas.

It's the first time I try those examples in qtourr-ex, by the way, I don't know we have such a nice demo. I tried the example before and after change to <<-, they all works for single plots, but they all gave errors for multiple plots, I don't know how it handles multiple plots, because in the example, three print function has been put in between the step function and start, only the first one, the qparallel, works on my laptop, other two plots don't get update at the same time. This issues exists even before the change. So I am not sure if the fix produce any problems on the tour demo, or it's just an issue produced some where else. Please change it back if it does affect the demo somehow, because I have already commited the change on this one one hour ago......

from cranvas.

It shouldn't matter that there are three print statements running three different tour displays. They were working a couple of days ago. I'll check them again.

On May 25, 2011, at 7:13 PM, tengfei wrote:

It's the first time I try those examples in qtourr-ex, by the way, I don't know we have such a nice demo. I tried the example before and after change to <<-, they all works for single plots, but they all gave errors for multiple plots, I don't know how it handles multiple plots, because in the example, three print function has been put in between the step function and start, only the first one, the qparallel, works on my laptop, other two plots don't get update at the same time. This issues exists even before the change. So I am not sure if the fix produce any problems on the tour demo, or it's just an issue produced some where else. Please change it back if it does affect the demo somehow, because I have already commited the change on this one one hour ago......

Reply to this email directly or view it on GitHub:
#22 (comment)

Di Cook
[email protected]

from cranvas.

I don't see this error message anymore. I think I can close this issue.

from cranvas.