fix plot labelling for >9 experiments #65
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Plot labelling for >9 experiments is currently broken because in the character representation of the experiment numbers, 9 comes after 10, 11, etc, which leads to a reordering of the boxes without a corresponding reordering of the labels. Example:
In the first panel the featureless learner moves to the end and is mislabelled.
This PR fixes the issue by zero-padding the strings generated from the numbers. Note that this is technically a workaround and will break if someone ever tries to show the results of more than 999,999,999 experiments in one plot.