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

Altair 4.1 "scale_factor" seems to do nothing #2191

Closed
zacharylawrence opened this issue Jun 3, 2020 · 4 comments
Closed

Altair 4.1 "scale_factor" seems to do nothing #2191

zacharylawrence opened this issue Jun 3, 2020 · 4 comments
Labels

Comments

@zacharylawrence
Copy link

zacharylawrence commented Jun 3, 2020

Using Altair 4.0, I'm able to adjust the resolution of an output PNG with scale_factor:

chart.save("filename.png", scale_factor=2.0)

In Altair 4.1, adjusting scale_factor doesn't seem to change the output resolution.

Is this possibly related to altair_saver?

As referenced here: https://altair-viz.github.io/user_guide/saving_charts.html#figure-size-resolution

@jakevdp jakevdp added the bug label Jun 3, 2020
@jakevdp
Copy link
Collaborator

jakevdp commented Jun 3, 2020

Altair 4.1 changed it's save to use altair_saver under the hood, so this bug is likely related to failing to plumb that value through correctly.

taralloc added a commit to taralloc/altair_saver that referenced this issue Dec 23, 2020
@jasonrwang
Copy link

@taralloc's PR fixes the resolution issue for me, but now I get weird spacing between characters.

New
image
Old
image

(I'm on Node v10.24.1, Altair 4.1, Altair Saver 0.5.0 with adjustment; macOS 11.2.3).

@baldwint
Copy link

baldwint commented May 7, 2022

Using @taralloc's branch also fixed it for me. I did not observe any strange spacing between characters.

node v15.4.0, altair 4.2.0, python 3.9.2, macOS 10.14.6.

@joelostblom
Copy link
Contributor

This has been fixed in Altair's development branch by switching to vl-convert for saving images. It is still an issue for altair_saver which is tracked in #2191

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants