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

DAMMviz() new keyword arguments for size flexibility #17

Merged
merged 4 commits into from
Mar 30, 2022

Conversation

BernhardAhrens
Copy link
Contributor

I added some keyword arguments to DAMMviz() to allow more flexible plotting on screens with different resolutions. Defaults are like before.
DAMMviz(width = 2200, height = 1600, fontsize = 30)

@AlexisRenchon
Copy link
Member

Hey @BernhardAhrens, thanks that's a good idea! The resolution issues has been bothering me.
I'll test and merge your PR today or tomorrow.

@AlexisRenchon AlexisRenchon merged commit 0d915f3 into CUPofTEAproject:master Mar 30, 2022
@AlexisRenchon
Copy link
Member

@BernhardAhrens should be up in v0.1.15 if you do ]update
btw, we started writing tests with @jamesafranke, hopefully we finish them soon and submit the MS to JOSS.

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