+
Skip to content

Conversation

edmundmiller
Copy link
Contributor

@edmundmiller edmundmiller commented Jul 28, 2025

The arm profile has been renamed to arm64 and now provides native ARM64 support using Wave containers instead of x86 emulation.

  • Replace arm profile with arm64 profile for native ARM64 execution
  • Add emulate_amd64 profile for users who need x86 emulation on ARM hardware
  • The arm64 profile works universally on Apple Silicon, AWS Graviton, and other ARM64 platforms
  • Users must update: -profile arm-profile arm64 (for native) or -profile emulate_amd64 (for emulation). This should only be users on apple silicon.

Hopefully users will be happy to migrate, it's a pretty big speed up.

Before

$ nextflow run . -profile test,docker,apple --outdir results 

Completed at: 07-May-2025 11:42:47
Duration    : 9m 24s
CPU hours   : 0.7 (0.1% failed)
Succeeded   : 213
Failed      : 2

After

$ nextflow run nf-core/rnaseq -profile test,docker,arm64,wave --outdir results
Completed at: 07-May-2025 12:15:55
Duration    : 4m 46s
CPU hours   : 0.4
Succeeded   : 213

edmundmiller and others added 2 commits July 28, 2025 13:53
BREAKING CHANGE: The `arm` profile has been renamed to `arm64` and now provides native ARM64 support using Wave containers instead of x86 emulation.

- Replace `arm` profile with `arm64` profile for native ARM64 execution
- Add `emulate_amd64` profile for users who need x86 emulation on ARM hardware
- The `arm64` profile works universally on Apple Silicon, AWS Graviton, and other ARM64 platforms
- Users must update: `-profile arm` → `-profile arm64` (for native) or `-profile emulate_amd64` (for emulation)

Fixes #3536

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Copy link

codecov bot commented Jul 28, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.22%. Comparing base (c3b528d) to head (b2bd61b).
⚠️ Report is 4 commits behind head on dev.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@edmundmiller edmundmiller linked an issue Jul 29, 2025 that may be closed by this pull request
@edmundmiller edmundmiller requested review from a team, mashehu and mirpedrol July 29, 2025 02:47
@edmundmiller edmundmiller self-assigned this Jul 29, 2025
@edmundmiller edmundmiller moved this to In progress in Seqera Containers Jul 29, 2025
@edmundmiller edmundmiller marked this pull request as ready for review July 29, 2025 02:51
@mashehu
Copy link
Contributor

mashehu commented Jul 29, 2025

Can we add a deprecation log message for people using the old profile?

Copy link
Member

@mirpedrol mirpedrol left a comment

Choose a reason for hiding this comment

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

LGTM!
While I agree that a deprecation message would be nice, I think it adds too much unnecessary code to the pipeline template, so I wouldn't add it here. We must make sure to mention this in the release blog post though!

@edmundmiller
Copy link
Contributor Author

That might have worked, but the new strict config parser won't allow it(according to @ewels ).

So just the warning message "there is no arm profile" will hopefully make them go look.

@edmundmiller edmundmiller enabled auto-merge August 19, 2025 12:48
@edmundmiller edmundmiller merged commit b926332 into dev Aug 19, 2025
79 of 101 checks passed
@edmundmiller edmundmiller deleted the rename-arm branch August 19, 2025 12:56
@github-project-automation github-project-automation bot moved this from In progress to Done in Seqera Containers Aug 19, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in nf-core infrastructure projects Aug 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Rename arm profile

4 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载