tlumip / swimr

Visualizations and Reports for SWIM
Other
0 stars 1 forks source link

Corrected percent difference calculations in compare scenarios Rmd template #32

Closed landisrm closed 5 years ago

landisrm commented 5 years ago

This resolves issue #93.

bstabler commented 5 years ago

I'm not sure if we want to restore the ability to get zones if using a database that doesn't have them added? @bettinardi - what do you think? Moving forward, do we want to assume that zone polygons will be in the database?

bettinardi commented 5 years ago

It will be tricky to work this in. My thoughts on preferred options:

  1. The code checks for zones, gets them if they are there and comes up with a different solution if it's an old data base where they don't exist
  2. (my vote if possible) - the old swimr still works with our existing runs. Issue - https://github.com/tlumip/tlumip/issues/95 works swimr into the SWIM run flow. My favorite idea is that all of these updates are kept in development until swimr is released in the SWIM run flow. That way it can help ensure that all of the required pre-reqs for swimr have been created.

The third cheapest option, is to carefully navigate what version of swimr is used with what version of the SWIM database - my least favorite option.

bstabler commented 5 years ago

I just talked with @bettinardi and we're going to do the following:

Once this is complete, @bettinardi can install the package and start using it with old and new swim runs. We're going to hold off on starting tlumip issue #95

landisrm commented 5 years ago

I committed changes to functions extract_zones, change_leaflet, and diff_leaflet in file zone_leaflet.R to allow for specifying whether to use built-in geography datasets or import from the database.

bstabler commented 5 years ago

this is done