google-research / rliable

[NeurIPS'21 Outstanding Paper] Library for reliable evaluation on RL and ML benchmarks, even with only a handful of seeds.

Home Page:https://agarwl.github.io/rliable

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Bug in plot_utils.py

zhefan opened this issue · comments

Hi,

In plot_utils.py, I think this line ought to be algorithms = list(point_estimates.keys())

algorithms = point_estimates.keys()

Otherwise, algorithms cannot be indexed in the next line.

Yes, it seems that the other plotting functions use list(point_estimates.keys()) but missed it here. Thanks for catching this .. there are tests for main library functions but not for the plotting functions. Will fix.

Thanks for fixing it!