Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update decimal places for psra ect_fatality, ectr_fatality #141

Closed
wkhchow opened this issue Oct 14, 2022 · 3 comments · Fixed by #150
Closed

update decimal places for psra ect_fatality, ectr_fatality #141

wkhchow opened this issue Oct 14, 2022 · 3 comments · Fixed by #150

Comments

@wkhchow
Copy link
Collaborator

wkhchow commented Oct 14, 2022

After discussion with @plesueur. Update fatality indicators from original 6dp to 10dp to capture more values.

@plesueur
Copy link

one thing to note @wkhchow, this will change the normalized risk scores, and therefore the normalized risk ratings.. Did we implement the jenks algorithm to calculate the thresholds for the rating in your database? If not, I'll need to do this externally before we propagate the dataset through the publishing process.

@wkhchow
Copy link
Collaborator Author

wkhchow commented Oct 14, 2022

changes applied in newest model_factory branch
3d77632
e7226f8
e44921f
385dbe1

@wkhchow
Copy link
Collaborator Author

wkhchow commented Oct 17, 2022

one thing to note @wkhchow, this will change the normalized risk scores, and therefore the normalized risk ratings.. Did we implement the jenks algorithm to calculate the thresholds for the rating in your database? If not, I'll need to do this externally before we propagate the dataset through the publishing process.

Jenks applied to the risk scores for sauid, csd and all hexgrid variations. Using ESRI ArcMap symbolization, and selecting entire data for sample to get correct jenks thresholds

@wkhchow wkhchow linked a pull request Feb 15, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants