rlcard
rlcard copied to clipboard
There is a bug in rlcard.utils.logger.py
Good day to you all! Recently I just got familiar with the RLCard package and I found a bug:
for the function : plot_curve(csv_path, save_path, algorithm) in the rlcard.utils.logger.py file, it doesn't have the right indent.
I think this function belongs to the Logger class in the same file. Please edit the file for it may cause confusion to the fresher coder(like me ) when they try this package.
@blackkingback Thanks for the feedback. I agree with you that it will be clearer to put the plot_curve in the Logger. We will update the code.
@blackkingback For this issue, I have moved the plot_curve function to utils.py to avoid any confusion. We still prefer to use a separate function instead of integrating the plotting into the logger. In this way, the logger is cleaner as it only logs.