-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Squeeze out space in profile plots #126
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other than that, LGTM!
Ahhh, I see what you mean, we’ll need to increase the size and font size, since increasing the size has the effect of making the font look smaller. I’ll try this out soon |
Alright, updated! Would be nice to hear feedback on these changes when people get a chance. I think it's looking near complete 🚀 |
Seems like you need to pass through the case name to the plotting functions. |
0c48110
to
698932a
Compare
Whoops, updated, hopefully this works. |
991ad9d
to
eec9571
Compare
eec9571
to
e0fb523
Compare
Going to merge this for now. I think we can do tweaking beyond this if people wish. |
bors r+ |
This PR updates the layout of the profile plots to squeeze out the lost white space. I also changed the layout to be 5 columns (it was previously 4). We can change it back if people prefer 4, most of the gained space was in the horizontal direction, so I think this actually preserves the look of the figures over sticking with 4, but I have no strong feelings about this.
Closes #124.