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

Question about documentation in probability_of_improvement #10

Closed
zhefan opened this issue Apr 17, 2022 · 2 comments
Closed

Question about documentation in probability_of_improvement #10

zhefan opened this issue Apr 17, 2022 · 2 comments

Comments

@zhefan
Copy link

zhefan commented Apr 17, 2022

Hi, I wonder if the documentation in probability_of_improvement function in metrics.py is wrong?
Specifically,

scores_x: A matrix of size (num_runs_x x num_tasks) where scores_x[m][n] represent the score on run n of task m for algorithm X.

scores_x: A matrix of size (`num_runs_x` x `num_tasks`) where scores_x[m][n]
)

Should scores_x[n][m] be the score on run n of task m for algorithm X?

Thanks.

@agarwl
Copy link
Collaborator

agarwl commented Apr 18, 2022

You are right -- I think it should be that score_x[n][m] is score on run n on task m around looking at the code here: https://github.com/google-research/rliable/blob/master/rliable/metrics.py#L92. Will fix.

@zhefan
Copy link
Author

zhefan commented Apr 18, 2022

Thanks for the clarification.

@zhefan zhefan closed this as completed Apr 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants