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

Log scale color mapping and arial default font #499

Merged
merged 2 commits into from
Dec 13, 2019
Merged

Conversation

banesullivan
Copy link
Member

This enables access to a log scale when mapping the scalars during plotting and also changes the default font to Arial because courier looked harsh to me

Using the dataset from pyvista/pyvista-support#83

No log scale:

download

With a log scale:

download

@banesullivan banesullivan added the enhancement Changes that enhance the library label Dec 13, 2019
@banesullivan
Copy link
Member Author

Also partially addressing the norm stuff in #122

Copy link
Member

@akaszynski akaszynski left a comment

Choose a reason for hiding this comment

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

Perfect. This was exactly what I was looking for on pyvista/pyvista-support#83.

@banesullivan
Copy link
Member Author

@akaszynski, you cool with changing the default font to Arial?

@akaszynski
Copy link
Member

That’s what paraview uses, right?

@banesullivan
Copy link
Member Author

I think so... my version of ParaView has that as the default though I could have set it a while back

@banesullivan banesullivan added this to the 0.23.0 milestone Dec 13, 2019
@akaszynski
Copy link
Member

I prefer courier, but then again I like Emacs, so I can be trusted. I’m good with Arial.

@banesullivan banesullivan merged commit 51bc5d1 into master Dec 13, 2019
@banesullivan banesullivan deleted the patch/scalar-bar branch December 17, 2019 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Changes that enhance the library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants