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

Force LVT contingency table metrics to use same contingency table structure #730

Open
emkemp opened this issue Mar 3, 2021 · 1 comment
Labels
DiscussionNeeded enhancement New feature or request

Comments

@emkemp
Copy link
Contributor

emkemp commented Mar 3, 2021

Presently each LVT contingency table metric (PSS, POD, CSI, etc) all store contingency table counts in their own, unique data structure. This wastes both computer memory and code, as brand new data structures and customized read/write subroutines must be written for each metric.

Proposal: Add a general contingency table structure that stores all table counts (hits, misses, false alarms, correct nulls), with support for stratification, and use this structure if any contingency table metric needs to be calculated. Leverage this structure as much as possible (e.g., writing and reading restarts, pulling data from this structure when it is time to calculate a particular metric).

This would be a intermediate term refactoring project (mostly because of conflicting demands on time).

@emkemp emkemp added enhancement New feature or request DiscussionNeeded labels Mar 3, 2021
@emkemp emkemp changed the title Force LVT contingency table metrics to use same contingency table elements Force LVT contingency table metrics to use same contingency table structure Mar 3, 2021
@emkemp
Copy link
Contributor Author

emkemp commented Jul 16, 2024

I still think we should do this. Someday....

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DiscussionNeeded enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant