Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update base image to 24.12 #553

Merged
merged 5 commits into from
Dec 31, 2024
Merged

update base image to 24.12 #553

merged 5 commits into from
Dec 31, 2024

Conversation

pstjohn
Copy link
Collaborator

@pstjohn pstjohn commented Dec 23, 2024

Attempts to update the base image to the most recent 24.12 release

@pstjohn
Copy link
Collaborator Author

pstjohn commented Dec 24, 2024

/build-ci

@trvachov
Copy link
Collaborator

Did you build-ci work given that this is a fork? I actually am not certain our ci will work on forks (...this is bad if it doesn't, since fork PRs should probably be the norm so that we dont have the branch explosion that we have today).

@trvachov
Copy link
Collaborator

/build-ci

@pstjohn
Copy link
Collaborator Author

pstjohn commented Dec 30, 2024

Did you build-ci work given that this is a fork? I actually am not certain our ci will work on forks (...this is bad if it doesn't, since fork PRs should probably be the norm so that we dont have the branch explosion that we have today).

Yeah it works on forks; that's pretty essential given that we're open source and not adding write-access external collaborators. IIUC that should be the recommended way to make PRs at this point, otherwise we clog up the main repo with a bunch of branches

@pstjohn
Copy link
Collaborator Author

pstjohn commented Dec 30, 2024

/build-ci

@pstjohn pstjohn enabled auto-merge (squash) December 30, 2024 21:59
Dockerfile Outdated Show resolved Hide resolved
@pstjohn
Copy link
Collaborator Author

pstjohn commented Dec 31, 2024

/build-ci

@pstjohn pstjohn merged commit 5bc7bdb into NVIDIA:main Dec 31, 2024
4 checks passed
@pstjohn
Copy link
Collaborator Author

pstjohn commented Dec 31, 2024

sudo-merging after CI passed due to folks being out of the office

trvachov added a commit that referenced this pull request Jan 1, 2025
## Summary
Un-xfails a geneformer H100 test.

## Details
After base image upgrade to pytorch fw 24.12
(#553) , H100 geneformer
issue is fixed.

## Usage and Testing
```python
pytest ./sub-packages/bionemo-geneformer/tests/bionemo/geneformer/test_model.py::test_geneformer_nemo1_v_nemo2_inference_golden_values
```
nvdreidenbach pushed a commit to nvdreidenbach/bionemo-framework that referenced this pull request Jan 2, 2025
Attempts to update the base image to the most recent 24.12 release

Signed-off-by: Danny <[email protected]>
nvdreidenbach pushed a commit to nvdreidenbach/bionemo-framework that referenced this pull request Jan 2, 2025
## Summary
Un-xfails a geneformer H100 test.

## Details
After base image upgrade to pytorch fw 24.12
(NVIDIA#553) , H100 geneformer
issue is fixed.

## Usage and Testing
```python
pytest ./sub-packages/bionemo-geneformer/tests/bionemo/geneformer/test_model.py::test_geneformer_nemo1_v_nemo2_inference_golden_values
```

Signed-off-by: Danny <[email protected]>
nvdreidenbach pushed a commit to nvdreidenbach/bionemo-framework that referenced this pull request Jan 2, 2025
Attempts to update the base image to the most recent 24.12 release

Signed-off-by: Danny <[email protected]>
nvdreidenbach pushed a commit to nvdreidenbach/bionemo-framework that referenced this pull request Jan 2, 2025
## Summary
Un-xfails a geneformer H100 test.

## Details
After base image upgrade to pytorch fw 24.12
(NVIDIA#553) , H100 geneformer
issue is fixed.

## Usage and Testing
```python
pytest ./sub-packages/bionemo-geneformer/tests/bionemo/geneformer/test_model.py::test_geneformer_nemo1_v_nemo2_inference_golden_values
```

Signed-off-by: Danny <[email protected]>
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.

2 participants