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

pygmt.set_display: Support setting image dpi and width globally #2448

Open
seisman opened this issue Mar 21, 2023 · 0 comments
Open

pygmt.set_display: Support setting image dpi and width globally #2448

seisman opened this issue Mar 21, 2023 · 0 comments
Labels
feature request New feature wanted

Comments

@seisman
Copy link
Member

seisman commented Mar 21, 2023

Description of the desired feature

The idea is to let set_display() have two more parameters dpi and width which can change the image dpi and width for all images displayed by the Figure.show() calls.

Some comments originally posted by @weiji14 in #529 (comment)

Personally I don't see a use-case for setting a global dpi or width (but happy to be proven wrong). My suggestion is that we only let this set_display function set the method and not do anything about the dpi or width. Haven't looked too closely, but I think most of the changes were made in ca0ecb3 and could be partially reverted there. Not doing anything about the dpi/width would also greatly reduce the diff on this PR and make it easier to review. Discuss first though.

Are you willing to help implement and maintain this feature?

Maybe

@seisman seisman added the feature request New feature wanted label Mar 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature wanted
Projects
None yet
Development

No branches or pull requests

1 participant