The issue has been solved

Issue a warning if using an Apple Silicon Mac under Rosetta #1362

JonnyBurger posted onGitHub

🎃 This issue is part of our Hacktoberfest campaign! 📚 Read more about Hacktoberfest here 🕛 This issue is currently assigned to @danmana and is not for taking

Node 14 uses Rosetta under M1, which makes Remotion 2x slower. But also newer versions like Node 16 could be running the x86 version on Apple Silicon.

A warning should be issued, ideally pointing to a new documentation article (Could be hosted under remotion.dev/docs/troubleshooting/rosetta), urging the user to upgrade

Hacktoberfest acceptance criteria

Write code that correctly identifies that the user has this problem and add it to the @remotion/renderer package, and run it once the package is imported.
Avoid false positives and false negatives
Add a new documentation page to the docs and describe how the user can fix this issue (upgrade from Node 14, if already on a higher version then prefix the command with arch -arm64)
Write a good error message and link the documentation page you posted.

@jonnyburger has funded $112.00 to this issue.

Submit pull request via IssueHunt to receive this reward.
Want to contribute? Chip in to this issue via IssueHunt.
Checkout the IssueHunt Issue Explorer to see more funded issues.
Need help from developers? Add your repository on IssueHunt to raise funds.

posted by issuehunt-app[bot] over 2 years ago

Hi, I'd like to take this.

Did I understand correctly that the warning message should be shown when the CPU model is Apple silicon (M1 or others from the family), but the architecture is x64 instead or arm64.

The recommendation would be:

if using Node 14 -> upgrade to node 16
if using Node 16 -> recommend using arch -arm64

posted by danmana over 2 years ago

@danmana Nice, good luck! Yes that is correct, if that also seems to you like the best way to do it, let's take this route.

posted by JonnyBurger over 2 years ago

Added a PR.

First time working with remotion, so I'm open for feedback. Note: I wasn't able to test the doc changes, docusaurus kept crashing ...

posted by danmana over 2 years ago

Thanks a lot, will give this a review tomorrow! Looking good! 🙌🏼

posted by JonnyBurger over 2 years ago

I managed to get docusaurus running and realized I added the docs in the wrong place and forgot to link to it in the sidebar. I updated the PR with this fix

posted by danmana over 2 years ago

@jonnyburger has rewarded $100.80 to @danmana. See it on IssueHunt