Skip to content
This repository has been archived by the owner on Apr 9, 2024. It is now read-only.

save chart using method "node" not running anymore with npm@9 #113

Closed
mhombrecher opened this issue Nov 27, 2022 · 4 comments
Closed

save chart using method "node" not running anymore with npm@9 #113

mhombrecher opened this issue Nov 27, 2022 · 4 comments

Comments

@mhombrecher
Copy link

mhombrecher commented Nov 27, 2022

Saving a chart as .png causes an "CalledProcessError" error when using method "node" and npm version is node@9

Reason is that npm has removed command: "npm bin" since version 9 which is used in _node.py (npm_bin)

@mhombrecher mhombrecher changed the title save chart using method "node" not running anymore with node@9 save chart using method "node" not running anymore with npm@9 Nov 27, 2022
@cotsuka
Copy link

cotsuka commented Jan 4, 2023

Confirming rolling back to npm < 9 seems to work.

aravindvnair99 added a commit to anmol098/waka-readme-stats that referenced this issue Jan 12, 2023
* NPM update removed [because of breaking changes introduced in NPMv9](altair-viz/altair_saver#113), NPMv8 is provided with container [by default](https://hub.docker.com/r/nikolaik/python-nodejs)

* Set `npm` to not go beyond v9 because of altair-viz/altair_saver#113

v9 has a breaking change for our dependency

Co-authored-by: Aravind Nair <[email protected]>
@aravindvnair99
Copy link

Hi team!

Any updates on this issue?

@jschaef
Copy link

jschaef commented Feb 2, 2023

This
anmol098/waka-readme-stats#337 (comment)

is working for me with npm --version 9.3.1

@joelostblom
Copy link
Member

Since Altair 5.2, the functionality of Altair Saver is now available in Altair via the vl-convert package. Most of the functionality has been available since 5.0, and the main addition in 5.2 was PDF export. See the docs on how to save charts for more details

We are going to archive this repo, so I'm closing all the open issues and PRs before doing so. Try out the new options for saving charts mentioned above and if you run into issues, please open an issue directly in the altair or vl-convert repo.

@joelostblom joelostblom closed this as not planned Won't fix, can't repro, duplicate, stale Mar 30, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants