I successfully create a plot using the following:
# suppose I have a p <- ggplot(data=df, ...) then the following works
# I get those two segments plotted correctly
p <- p + geom_segment(aes(x=1,y=103,xend=1,yend=107))
p <- p + geom_segment(aes(x=5,y=103,xend=5,yend=107))
However if I do:
values <- c(1, 5)
for (i in values) {
p <- p + geom_segment(aes(x=i,y=103,xend=i,yend=107))
}
It doesn't work, only the last segment is created. Can anyone advice what's wrong here?
An alternative approach would be to avoid using a loop at all. You can pack your segment data up in a separate data.frame from your main data and use aes() to plot everything at once like so:
It has to do with the lazy evaluation of the
aes()
values. You are binding to the variablei
but not actually doing anything with it in the loop. The mappings aren't resolved till you actuallyprint(p)
. Essentially this means they are all being bound toi
and after the loop exits,i
will have the value it had during the final loop.So the problem really is you shounld't be using
aes()
here as you don't really want active binding. Just set thex
andxend
values outside theaes()
. (And since they
's are constant they should be outside theaes()
as well).