Skip to content

Bump NeMo and support cuhyena LI#1027

Closed
farhadrgh wants to merge 9 commits intomainfrom
farhadr/cuhyena_li
Closed

Bump NeMo and support cuhyena LI#1027
farhadrgh wants to merge 9 commits intomainfrom
farhadr/cuhyena_li

Conversation

@farhadrgh
Copy link
Collaborator

Description

Type of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactor
  • Documentation update
  • Other (please describe):

CI Pipeline Configuration

Configure CI behavior by applying the relevant labels:

Note

By default, the notebooks validation tests are skipped unless explicitly enabled.

Authorizing CI Runs

We use copy-pr-bot to manage authorization of CI
runs on NVIDIA's compute resources.

  • If a pull request is opened by a trusted user and contains only trusted changes, the pull request's code will
    automatically be copied to a pull-request/ prefixed branch in the source repository (e.g. pull-request/123)
  • If a pull request is opened by an untrusted user or contains untrusted changes, an NVIDIA org member must leave an
    /ok to test comment on the pull request to trigger CI. This will need to be done for each new commit.

Usage

TODO: Add code snippet

Pre-submit Checklist

  • I have tested these changes locally
  • I have updated the documentation accordingly
  • I have added/updated tests as needed
  • All existing tests pass successfully

Signed-off-by: Farhad Ramezanghorbani <farhadr@nvidia.com>
@copy-pr-bot
Copy link

copy-pr-bot bot commented Aug 6, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Signed-off-by: Farhad Ramezanghorbani <farhadr@nvidia.com>
@farhadrgh farhadrgh force-pushed the farhadr/cuhyena_li branch from bdacfb6 to a20bd75 Compare August 7, 2025 18:39
@farhadrgh
Copy link
Collaborator Author

/ok to test a20bd75

@codecov-commenter
Copy link

codecov-commenter commented Aug 7, 2025

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.29%. Comparing base (53a81c8) to head (f571bae).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...ackages/bionemo-evo2/src/bionemo/evo2/run/train.py 0.00% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1027   +/-   ##
=======================================
  Coverage   81.28%   81.29%           
=======================================
  Files         152      152           
  Lines       10271    10271           
=======================================
+ Hits         8349     8350    +1     
+ Misses       1922     1921    -1     
Files with missing lines Coverage Δ
...ackages/bionemo-evo2/src/bionemo/evo2/run/train.py 14.16% <0.00%> (ø)

... and 1 file with indirect coverage changes

Signed-off-by: Farhad Ramezanghorbani <farhadr@nvidia.com>
Signed-off-by: Farhad Ramezanghorbani <farhadr@nvidia.com>
@farhadrgh farhadrgh enabled auto-merge August 13, 2025 15:41
@farhadrgh farhadrgh changed the title [WIP] Bump NeMo and support cuhyena LI Bump NeMo and support cuhyena LI Aug 13, 2025
@farhadrgh
Copy link
Collaborator Author

/ok to test f571bae

Copy link
Collaborator

@trvachov trvachov left a comment

Choose a reason for hiding this comment

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

Might be a naming issue

Signed-off-by: Farhad Ramezanghorbani <farhadr@nvidia.com>
@farhadrgh farhadrgh requested a review from ohadmo as a code owner August 14, 2025 21:02
Signed-off-by: Farhad Ramezanghorbani <farhadr@nvidia.com>
Signed-off-by: Farhad Ramezanghorbani <farhadr@nvidia.com>
Signed-off-by: Farhad Ramezanghorbani <farhadr@nvidia.com>
Signed-off-by: Farhad Ramezanghorbani <farhadr@nvidia.com>
@farhadrgh farhadrgh closed this Aug 15, 2025
auto-merge was automatically disabled August 15, 2025 16:03

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants