PNHP / COA

Pennnsylvania Conservation Opportunity Area Tool development scripts
GNU General Public License v3.0
2 stars 1 forks source link

Add probability values to report text #122

Open ChristopherTracey opened 6 years ago

ChristopherTracey commented 6 years ago

Before the SGCN table we have a line of text that should be changed to something like this:

“The table below shows 13 Species of Greatest Conservation Need (SGCN) that have been identified as either occurring (High) or potentially occurring (Medium) within your area of interest.”

Additionally, We should find a way to define these occurrence probabilities need to be defined somehow.

ChristopherTracey commented 6 years ago

Also should try to to officially deal with this in #88.

This is also referenced in #96, in regard to whether or not we include low values in the table, but do not provide actions for them. The current solution is to list these below the table, which works fairly well. I'm not sure if we want the table to be bigger.

ChristopherTracey commented 6 years ago

As far a definitions for the Occurrence Probs, how about this for a start?

ChristopherTracey commented 6 years ago

@dyeany, we should talk about this tomorrow as we cover the mapping issues.

ChristopherTracey commented 6 years ago

Integrated definitions into the RNW as above, but I think we need to change the definitions up to be easier to understand. We should consider getting rid of the text that reads 'upper third', etc.

ChristopherTracey commented 6 years ago

For reference: https://github.com/zonination/perceptions

ChristopherTracey commented 6 years ago

C/D want these to replace High and Medium: • Confirmed - The species has been documented within the area of interest. • Probable - The species has not been documented within the area of interest, but species distribution models or other data sources indicate strong potential for its occurrence.

Still waiting on what to do about Low.

ChristopherTracey commented 6 years ago

Ugh... This is going to be more of a mess than I thought to rewrite into the code.

ChristopherTracey commented 6 years ago

Closed the #182 issue as it was a duplicate.

ChristopherTracey commented 6 years ago

Change the values on the report to 'Confirmed' and 'Probable'. Still have to upgrade code to use the new values in the table as well as change data frame names in the code.

ChristopherTracey commented 6 years ago

Began to replace values this at the SQL database level using the following code: Update lu_sgcnXpu_all Set OccProb = replace(OccProb, 'High', 'Confirmed') and Update lu_sgcnXpu_all Set OccProb = replace(OccProb,'Medium','Probable')

ChristopherTracey commented 6 years ago

Updated code in #212. Everything seems to work.

@memoore and @dyeany, we need to update the geodatabases and the downstream tools to use these new values.