Skip to content
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

Feature: Adding bundle for ProbNum2025 #155

Merged
merged 7 commits into from
Oct 17, 2024
Merged

Feature: Adding bundle for ProbNum2025 #155

merged 7 commits into from
Oct 17, 2024

Conversation

philipphennig
Copy link
Collaborator

Feature: style file for ProbNum

This PR adds the bundles, figsizes, fonts, and fontsizes for ProbNum 2025.

The numbers are based on the style file. Note: That style isn't yet published in the CfP, as I am actively developing it. My goal is in fact to include a link to the tueplots bundle provided by this PR in the style file, and explicitly ask submitting authors to use the bundle. To make this possible I am first submitting this PR, and will then subsequently publish the style file.

Minor changes may be needed in the future, as the style file is yet to be sent out for feedback.

Copy link
Owner

@pnkraemer pnkraemer left a comment

Choose a reason for hiding this comment

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

Great - I am looking forward to this!

Could you also add corresponding test-cases for the bundles, figsizes, fonts, and fontsizes? Concrete instructions should be in the online documentation. If you can't find them, let me know and I will help.

I left two comments (respectively three, but one is a duplicate) in the diff. I am wondering what you think about those.

tueplots/bundles.py Outdated Show resolved Hide resolved
tueplots/figsizes.py Outdated Show resolved Hide resolved
tueplots/figsizes.py Outdated Show resolved Hide resolved
@philipphennig
Copy link
Collaborator Author

Sorry that I missed the test cases. And I thought I had read the docs!

Well, they are in now.

Copy link
Owner

@pnkraemer pnkraemer left a comment

Choose a reason for hiding this comment

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

Perfect, thank you! I'll release the new code so you can try the setup when finalising the style file. If we need an update, let me know.

@pnkraemer pnkraemer merged commit 0f970e8 into main Oct 17, 2024
16 checks passed
@pnkraemer pnkraemer deleted the PH branch October 17, 2024 12:58
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