-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add performance index #149
base: main
Are you sure you want to change the base?
Conversation
Please note that formatting did some additional changes in gsd/evaluation.py |
Tests still to be added - any suggestions? |
@a-mosquito Can you have a look at this? In particular considering the changes that you made for error calculation. Do you see a way this could be integrated better with the other error calculation stuff? |
It should be possible to integrate that to the syntax introduced by #126. We could model the performance index calculation as a chain of transformations & aggregations:
Some considerations for the practical side of implementing this:
|
Added performance index calculation as discussed in #122