LandSciTech / ccviR

Implement NatureServe climate change vulnerability index in R
https://landscitech.github.io/ccviR/
Other
2 stars 1 forks source link

Document need to run spatial analysis when loading data from previous assessment #135

Closed adricaswell closed 4 months ago

adricaswell commented 4 months ago

If data is loaded from previous assessment without rerunning spatial analysis, user will not be able to see maps. Additionally, if a report is generated from a previous assessment, it will include the raw spatial data inputs, not the processed ones (see example below). To ensure the user uses the app properly, explain this in the documentation.

Screenshot 2024-02-29 104759 Screenshot 2024-02-29 104854
adricaswell commented 4 months ago

This need to re-run spatial analysis becomes more of an issue if a user changed their spatial vulnerability question answers from the default. Re-running spatial analysis would effectively erase their progress. To ensure users do not lose their data, this issue should either be explicitly documented or, if possible, fixed somehow.

adricaswell commented 4 months ago

The error message in the code block below appears when trying to load map/raster in exposure results tab without having run spatial analysis.

doSpatial restore
spat out_data
Warning: Error in renderTable: climate readme file does not contain breaks for ccei
  100: stop
   99: renderTable
   98: func
   85: renderFunc
   84: output$tbl_ccei
    3: runApp
    2: print.shiny.appobj
    1: <Anonymous>

Tables appear to be displayed properly. Running spatial analysis resolves this error.

This error does not appear when trying to load the spatial vulnerability question maps. The maps simply do not load.

see24 commented 4 months ago

Make it run spatial analysis if it had already been run and then updating the spatial vuln qs from the csv