geogugal/quickmapr

An R package for quickly mapping and navigating spatial data

quickmapr's Issues

check ordering, raster may be messed up

New functionality: have select select based on poly, not just point

Clean up i() so that it only returns data for the selected object

See #14

rasters with colormaps do not plot correctly on zoom

Appears to be a problem with the colormaps and the values currently plotted. Also, plot sill seems to be taking a long time on tighter zooms. Can test this with the VT lulc from VCGIS.

Ideas:

• re-calc color map for curretnly displayed values
• mask gets color maps right so could mask raster data and plot that instead. Need to check timing on this.

Vignette

Finish up a vignette for push to CRAN

i needs to return column names for data.

Test Suite

Build test suite for push to CRAN

Speed up basemap

Currently have it working with jpeg, but with high res download and interactiviity is slow. Low res sucks on zoom.

Need to try to implement the download on zoom again. Res will increase as extent decreases.

See #4

Color and Fill controls

work on better methods for changing colors and color maps of raster

allow i() to return results from multiple layers.

basemap

Currently basemap is an option on qmap() and new basemaps get downloaded on zoom. This has the benefit of better resolution when you zoom, but is SLOW.

Try having separate workflow for getting a single image with get_basemap, then alter plot to deal with multiple bands. Maybe keep basemap option on qmap but use single image only.

add some basic checking/custom errors in qmap

regenerate readme?

last run all examples broke it looks like

Please remove dependencies on **rgdal**, **rgeos**, and/or **maptools**

This package depends on (depends, imports or suggests) raster and one or more of the retiring packages rgdal, rgeos or maptools (https://r-spatial.org/r/2022/04/12/evolution.html, https://r-spatial.org/r/2022/12/14/evolution2.html). Since raster 3.6.3, all use of external FOSS library functionality has been transferred to terra, making the retiring packages very likely redundant. It would help greatly if you could remove dependencies on the retiring packages as soon as possible.

Zoom in zooming twice on initial zoom

Only notice with basemap, but haven't looked widely yet.

Test without basemap and fix

review coverage on tests

testing is only at ~65%. Figure out what needs to be added.

Add tests for select and measure (when those functions are complete).

Raster Draw Order

Need to check for raster and figure out a way to draw these. One raster = draw raster first. Multiple may just have to return a warning.

add a measure tool

Add check and test that input on zoom, pan, etc is a qmap object

trying to zi in on sp object fails ungracefully.

a few thoughts...

had a look over, just a few quick thoughts

• i'm pretty sure you don't need @method in your function definitions like print.qmap
• minor point but maybe link to functions within your docs, like \code{\link{qmap}} instead of \code{qmap}

when using qmap object as input to qmap, options (i.e. colors) get reset to defaults

Look at readme for example. Colors set fill to blue, use that object as input for basemap and the defaults are reset. Need to pull these from existing settings on qmap object if not set upon subsequent use.

Zooming and panning slow with modestly sized sp objects

Zooming with zoom seems to have some performance issues, especially with polys.

Initial draw is fast enough, perhaps try to rework zoom to just be a fresh draw based on a new extent. would require creating a quickmapr object and then passing that object plus a new extent for each invocation of zi

Method for changing symbology: points and lines

Change to codecov

Add select() function that allows selection of individual point, lines or polygons and returns those as sp objects

Add select function, as suggest in #14