Skip to content

Conversation

@StephenNneji
Copy link
Contributor

No description provided.

Copy link
Collaborator

@DrPaulSharp DrPaulSharp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This does the job well, I have a couple of thoughts based on my graph here:

image

  • Is there a way to use more of the canvas - there's a fair bit of white space at the top and to the right?
  • The 1e-6" labels on the left hand side do not clearly match up to a row of plots. Is it possible to align them with the centre of the corresponding row?
  • Some of the axis values (e.g., x axis of bottom left plot) show the numbers running into each other. Can you do anything to prevent this?

@DrPaulSharp
Copy link
Collaborator

I'm also finding it runs very slowly for me on my machine using pycharm in windows, but I think that problem is on my end.

@StephenNneji
Copy link
Contributor Author

StephenNneji commented May 14, 2025

Is there a way to use more of the canvas - there's a fair bit of white space at the top and to the right?

I have made the figure size closer to a square to reduce white space on right but I can't do more as compacting the layout destroys the look of the plot for large number of params and affects performance

The 1e-6" labels on the left hand side do not clearly match up to a row of plots. Is it possible to align them with the centre of the corresponding row?

No easy way to address but I have found a workaround.

Some of the axis values (e.g., x axis of bottom left plot) show the numbers running into each other. Can you do anything to prevent this?

Can't do anything about this as far as I know

@StephenNneji StephenNneji merged commit 37c0696 into RascalSoftware:main May 14, 2025
7 checks passed
@StephenNneji StephenNneji deleted the improve_corner branch September 3, 2025 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants