Skip to content
This repository has been archived by the owner on Jul 20, 2020. It is now read-only.

Graph in the README should be a scatter plot. #17

Open
halflings opened this issue Nov 24, 2018 · 1 comment
Open

Graph in the README should be a scatter plot. #17

halflings opened this issue Nov 24, 2018 · 1 comment

Comments

@halflings
Copy link

Really nitpicky remark, but it doesn't make sense to interpolate eval results between two different models, so a line plot is not appropriate (the wild drops/increases in loss make it hard to read as well). A scatter plot would convey the idea better I think!

@Motorrat
Copy link
Contributor

You have a point on interpolation meaning. However the model are combined into ensembles so there might be some meaning to that too.
Yet visually I like the line plot and had only positive feedback so far. On the contrary, a scatter plot version didn’t look as illustrative.
You’re welcome to submit your version of this plot, or any other illustration. It would be a great improvement to have more illustrations here.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants