Billingegroup / bg-mpl-stylesheets

Matplotlib style sheets
Other
4 stars 27 forks source link

fixes issue #32 #33

Closed pliu2781 closed 2 years ago

pliu2781 commented 2 years ago

fixes #32 with line if find_executable('latex'):

sbillinge commented 2 years ago

Thanks @pliu2781

What does the output look like if there is not latex installed or the font is missing (i.e., if the if statement is not entered)?

pliu2781 commented 2 years ago

For the example plot.py file, if latex isn't installed:

image

If latex is installed:

image

sbillinge commented 2 years ago

it looks ok. But is the Y-axis label lost?

On Mon, Oct 25, 2021 at 3:09 PM pliu2781 @.***> wrote:

For the example plot.py file, if latex isn't installed:

[image: image] https://user-images.githubusercontent.com/44211772/138755155-e9c867a5-89bd-4412-9454-4327823483b0.png

If latex is installed:

[image: image] https://user-images.githubusercontent.com/44211772/138755297-29aefd42-5f67-414b-9e7a-b6ce07306b93.png

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Billingegroup/bg-mpl-stylesheets/pull/33#issuecomment-951224917, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABAOWULGA73F33CBNON4LELUIWTQDANCNFSM5GV3F7YA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

-- Simon Billinge Professor, Columbia University Physicist, Brookhaven National Laboratory

pliu2781 commented 2 years ago

The y-axis label is cutoff, but it comes back if I use plt.tight_layout()

image

sbillinge commented 2 years ago

what do you think we should do? What happens to the one with the latex labels if you add in that code? Please come up with a suggestion for proceeding that makes the most sense....

Thanks,

S

On Mon, Oct 25, 2021 at 3:30 PM pliu2781 @.***> wrote:

The y-axis label is cutoff, but it comes back if I use plt.tight_layout()

[image: image] https://user-images.githubusercontent.com/44211772/138758107-8c1f22ae-edcc-491e-aac2-5d646b279208.png

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Billingegroup/bg-mpl-stylesheets/pull/33#issuecomment-951241058, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABAOWUOKD7VZ6246JWWRZTTUIWV6PANCNFSM5GV3F7YA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

-- Simon Billinge Professor, Columbia University Physicist, Brookhaven National Laboratory

pliu2781 commented 2 years ago

With latex, the output looks like this:

image

I think we should keep the plt.tight_layout() line in the example plot.py. Other than that, the issue should be resolved.