indexing allowed for points and lines

[lindsayplatt]

Does not label x axis as "Index" and y with range. Works for logged axes and keeps user-specified axis labels. Also works with dates and other classes. It also works for lines.

See #245

Originally, I had implemented this using xy.coords since that is what points.default and lines.default call, but if coerces data into numeric.

gs <- gsplot()  %>%
    points(1:10, xlab="my points", log="y")
gs

[coveralls]

Coverage increased (+0.1%) to 82.241% when pulling a1a239c on lindsaycarr:indexLost into e839398 on USGS-R:master.

[jiwalker-usgs]

I think this is fine, I'm just wondering if we will have an issue with functions like this going forward.

[jiwalker-usgs]

I think this should maybe be in function_args rather than here. I worry that adding special cases in set_args will complicate this function beyond where it is now.

[jordansread]

I agree w/ what @jiwalker-usgs said. this looks fine to me too, but we should (as in the past) just be able to pass the args on to points w/ y=NULL and have that function handle it appropriately.

[lindsayplatt]

@jread-usgs have points.gsplot handle it appropriately or graphics::points? The issue seems to be that we never pass it to points.default which calls plot.xy and handles this issue of y=NULL.

[jordansread]

I meant graphics::points, but I guess maybe this never worked correctly then?

[coveralls]

Coverage increased (+0.2%) to 82.329% when pulling d7e1ab0 on lindsaycarr:indexLost into e839398 on USGS-R:master.

[lindsayplatt]

I guess I'm struggling with the override function and how UseMethod works. It should get to points.default somehow, which handles the xy indexing https://svn.r-project.org/R/trunk/src/library/graphics/R/points.R