Closed Louis-Dupont closed 1 year ago
Maybe let's use term "Support" instead of "Frequency", because it's a bit (to me) misleading.
Second - We have inconsistent names on Y axis. It's "Class" on the first plot and "Class Names" in the second. Pro tip: Let's have some StringConstants class with pre-defined string literals for axis names and use them instead of hard-coding same names everywhere?
Second - We have inconsistent names on Y axis. It's "Class" on the first plot and "Class Names" in the second. Pro tip: Let's have some StringConstants class with pre-defined string literals for axis names and use them instead of hard-coding same names everywhere?
Good catch, I created a ticket to add these literals in the plot_options
To me there is still a few minor issues to address remaining. See inline comments
Example to show the difference between train/val