Skip to content

Remove recommendation to use Intel-based Conda installation#246

Merged
victorlin merged 2 commits intomasterfrom
victorlin/remove-conda-apple-silicon-faq
May 30, 2025
Merged

Remove recommendation to use Intel-based Conda installation#246
victorlin merged 2 commits intomasterfrom
victorlin/remove-conda-apple-silicon-faq

Conversation

@victorlin
Copy link
Member

@victorlin victorlin commented Apr 1, 2025 •
edited
Loading

Description of proposed changes

Replaced by another FAQ entry that describes the previous recommendation and why it was removed.

Related issue(s)

Follow-up to nextstrain/cli#379 (comment)

Checklist

@victorlin victorlin self-assigned this Apr 1, 2025
@victorlin victorlin force-pushed the victorlin/remove-conda-apple-silicon-faq branch from b9faab0 to b265fc8 Compare April 1, 2025 16:17
@victorlin victorlin mentioned this pull request Apr 1, 2025
Closed
4 tasks
genehack
genehack approved these changes Apr 1, 2025
View reviewed changes
Copy link
Contributor

@genehack genehack left a comment

Choose a reason for hiding this comment

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

<3 a PR that's all deletions.

@tsibley
Copy link
Contributor

tsibley commented Apr 1, 2025

It might be nice to leave a new (short) FAQ that explains that yes, we did use to recommend osx-64 even on ARM but now recommend osx-arm64.

victorlin added 2 commits May 27, 2025 14:42
@victorlin
Remove warning on using Apple silicon version of Conda
d4301a1 
Improvements have been made by the Bioconda community to support more
osx-arm64-native packages. I've tested that the example ambient setup
command now works without any problems using an Apple silicon version of
Micromamba.
@victorlin
Remove recommendation to use Intel-based Conda installation
67b39a1 
Replaced by another FAQ entry that describes the previous
recommendation and why it was removed.
@victorlin victorlin force-pushed the victorlin/remove-conda-apple-silicon-faq branch from b265fc8 to 67b39a1 Compare May 27, 2025 22:07
@victorlin victorlin changed the base branch from master to victorlin/remove-pangolin-install May 27, 2025 22:07
@victorlin
Copy link
Member Author

victorlin commented May 27, 2025

It might be nice to leave a new (short) FAQ that explains that yes, we did use to recommend osx-64 even on ARM but now recommend osx-arm64.

Added as (preview):

What happened to the recommendation to use an Intel-based Conda installation?

Background: computers with Apple silicon hardware have two options for Conda subdir: osx-arm64 (native) and osx-64 (Intel-based, requires Rosetta 2 emulation).

When Apple silicon was first released, few Conda packages supported osx-arm64. This meant that the example conda create command provided in the Nextstrain docs would fail to resolve packages when using an osx-arm64 Conda installation. We had suggested using an osx-64 Conda installation as an easy workaround for most users, at the cost of not taking full advantage of the Apple silicon architecture.

Now that there is wide support for osx-arm64 packages, we have removed the workaround.

@victorlin victorlin changed the title Remove FAQ entry on osx-arm64 Conda incompatibilities Remove recommendation to use Intel-based Conda installation May 30, 2025
Base automatically changed from victorlin/remove-pangolin-install to master May 30, 2025 22:00
@victorlin victorlin merged commit 8f48ba5 into master May 30, 2025
3 checks passed
@victorlin victorlin deleted the victorlin/remove-conda-apple-silicon-faq branch May 30, 2025 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@genehack genehack genehack approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@victorlin victorlin

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@victorlin @tsibley @genehack

Comments