-
Notifications
You must be signed in to change notification settings - Fork 2
Metrics #13
base: master
Are you sure you want to change the base?
Metrics #13
Conversation
Hello @cjayross! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2019-07-08 09:56:19 UTC |
I added the citations that I had available. I think there is a couple that are still missing. |
Addresses #4
This adds a total 113* different metrics that can be loaded by using the function
load_metric
defined inmetric.py
.There are currently two issues to be addressed, the first is that the metrics aren't organized and unsearchable; meaning that a user has to manually go into the file where it's defined to actually discover what the metric actually looks like. The second issues is that citations need to be added to each metric that is defined.
* Now 115