-
Notifications
You must be signed in to change notification settings - Fork 23
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
Error installing dependencies #97
Comments
Got the same error on a MacBook M1 running 13.5.2 while bringing in the current package.json to update an older branch. Took out the canvas dependency and it seems to be what is causing the issue, maybe a MacOS incompatibility? Examples can still run through yarn start unaffected. |
Well this seems like a real nightmare. The |
I tried a few things in those threads but none seems to work for me. Though someone did mention an alternative that supports both Intel and ARM chips: https://www.npmjs.com/package/@napi-rs/canvas. |
Ah, I've actually worked quite a bit with https://github.com/Automattic/node-canvas/wiki/Installation:-Mac-OS-X @lindapaiste is there a way to temporarily remove the tests that require this and revisit later or will that cause other problems? |
@shiffman I'll take a look. |
Without this package we cannot run end-to-end tests where we provide the model with an actual image and check that we got the correct result. So that's a bummer. But....even with this package I was still running into a lot of other problems with using browser code in Node. I think that our long term plan might be that we do certain end-to-end tests in a browser environment instead of trying to polyfill everything. We would use Jest for unit tests of specific logic. The bodypose tests will fail when removing |
This makes sense to me, @lindapaiste what do you think about pulling out the |
I am trying to run the
yarn
command to install the dependencies but got this error.It only appears on my M1 MacBook with macOS 14.3 and not on my Windows machine. The error disappeared when I reverted the changes of #82.
The text was updated successfully, but these errors were encountered: