Skip to content

Fix esm2 lora inference#996

Merged
yzhang123 merged 3 commits into
mainfrom
fix_esm2_lora_inference
Jul 24, 2025
Merged

Fix esm2 lora inference#996
yzhang123 merged 3 commits into
mainfrom
fix_esm2_lora_inference

Conversation

@yzhang123
Copy link
Copy Markdown
Collaborator

@yzhang123 yzhang123 commented Jul 22, 2025

Description

fix esm2 lora inference

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

@yzhang123
Copy link
Copy Markdown
Collaborator Author

/ok to test 3943a97

@yzhang123 yzhang123 enabled auto-merge July 22, 2025 20:35
Comment thread sub-packages/bionemo-esm2/src/bionemo/esm2/scripts/infer_esm2.py Outdated
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 22, 2025

Codecov Report

❌ Patch coverage is 91.66667% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.23%. Comparing base (c5b35a8) to head (7047f05).
⚠️ Report is 125 commits behind head on main.

Files with missing lines Patch % Lines
...ionemo-esm2/src/bionemo/esm2/scripts/infer_esm2.py 91.66% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #996      +/-   ##
==========================================
+ Coverage   83.18%   83.23%   +0.04%     
==========================================
  Files         148      148              
  Lines        9736     9758      +22     
==========================================
+ Hits         8099     8122      +23     
+ Misses       1637     1636       -1     
Files with missing lines Coverage Δ
...ionemo-esm2/src/bionemo/esm2/scripts/infer_esm2.py 67.74% <91.66%> (+8.58%) ⬆️

... and 1 file with indirect coverage changes

Comment thread sub-packages/bionemo-esm2/src/bionemo/esm2/scripts/infer_esm2.py Outdated
Signed-off-by: Yang Zhang <yangzhang@nvidia.com>
@yzhang123
Copy link
Copy Markdown
Collaborator Author

/ok to test 3943a97

@trvachov trvachov mentioned this pull request Jul 24, 2025
5 tasks
Copy link
Copy Markdown
Collaborator

@jstjohn jstjohn left a comment

Choose a reason for hiding this comment

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

🚢

@yzhang123 yzhang123 added this pull request to the merge queue Jul 24, 2025
Merged via the queue into main with commit acb6f64 Jul 24, 2025
26 checks passed
@yzhang123 yzhang123 deleted the fix_esm2_lora_inference branch July 24, 2025 18:46
edawson pushed a commit that referenced this pull request Jul 28, 2025
### Description
fix esm2 lora inference

### Type of changes
<!-- Mark the relevant option with an [x] -->

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

### CI Pipeline Configuration
Configure CI behavior by applying the relevant labels:

-
[SKIP_CI](https://github.com/NVIDIA/bionemo-framework/blob/main/docs/docs/user-guide/contributing/contributing.md#skip_ci)
- Skip all continuous integration tests
-
[INCLUDE_NOTEBOOKS_TESTS](https://github.com/NVIDIA/bionemo-framework/blob/main/docs/docs/user-guide/contributing/contributing.md#include_notebooks_tests)
- Execute notebook validation tests in pytest
-
[INCLUDE_SLOW_TESTS](https://github.com/NVIDIA/bionemo-framework/blob/main/docs/docs/user-guide/contributing/contributing.md#include_slow_tests)
- Execute tests labelled as slow in pytest for extensive testing

> [!NOTE]
> By default, the notebooks validation tests are skipped unless
explicitly enabled.

#### Authorizing CI Runs

We use
[copy-pr-bot](https://docs.gha-runners.nvidia.com/apps/copy-pr-bot/#automation)
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
<!--- How does a user interact with the changed code -->
```python
TODO: Add code snippet
```

### Pre-submit Checklist
<!--- Ensure all items are completed before submitting -->

 - [ ] 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: Yang Zhang <yangzhang@nvidia.com>
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.

5 participants