Commit 85d0c16
authored
fix: xfail AMPLIFY FSDP2 tests and remove stale ESM2 HF FSDP2 xfail (#1564)
## Nightly CI Fix (2026-04-26)
Fixes nightly CI failure in `unit-tests-recipes.yml`
([run](https://github.com/NVIDIA/bionemo-framework/actions/runs/24953246928)):
### 1. `esm2_accelerate_te` —
`test_accelerate_amplify.py::test_te_with_fsdp2_config` FAILED
**Root cause:** The AMPLIFY model on HuggingFace Hub
(`nvidia/AMPLIFY_120M`) does not implement `get_input_embeddings()`,
which the accelerate FSDP2 prepare path now requires
(`transformers>=5.6`).
The previous fix (#1563) added `get_input_embeddings` to a local copy of
`amplify_te.py`, but the test loads the model from HuggingFace Hub via
`AutoModelForMaskedLM.from_config(config, trust_remote_code=True)`, so
the local fix had no effect.
**Fix:** Mark `test_te_with_fsdp2_config` and
`test_te_with_fsdp2_config_two_gpus` as `xfail(strict=True)` until the
upstream AMPLIFY model is updated.
### 2. `esm2_accelerate_te` — `test_hf_with_fsdp2_config` XPASSED
**Root cause:** The `xfail` for BIONEMO-3331 is stale — ESM2 HF FSDP2
tests now pass with the current container.
**Fix:** Remove the `xfail` decorators from `test_hf_with_fsdp2_config`
and `test_hf_with_fsdp2_config_two_gpus`.
---
*Automated fix by svc-bionemo nightly CI monitor.*
Signed-off-by: svc-bionemo <267129667+svc-bionemo@users.noreply.github.com>
Co-authored-by: svc-bionemo <267129667+svc-bionemo@users.noreply.github.com>1 parent 34aad73 commit 85d0c16
2 files changed
Lines changed: 10 additions & 4 deletions
File tree
- bionemo-recipes/recipes/esm2_accelerate_te/tests
Lines changed: 10 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
20 | 20 | | |
21 | 21 | | |
22 | 22 | | |
| 23 | + | |
| 24 | + | |
23 | 25 | | |
24 | 26 | | |
25 | 27 | | |
| |||
39 | 41 | | |
40 | 42 | | |
41 | 43 | | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
42 | 48 | | |
43 | 49 | | |
44 | 50 | | |
| |||
56 | 62 | | |
57 | 63 | | |
58 | 64 | | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
59 | 69 | | |
60 | 70 | | |
61 | 71 | | |
| |||
Lines changed: 0 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
13 | 13 | | |
14 | 14 | | |
15 | 15 | | |
16 | | - | |
17 | | - | |
18 | 16 | | |
19 | 17 | | |
20 | 18 | | |
| |||
49 | 47 | | |
50 | 48 | | |
51 | 49 | | |
52 | | - | |
53 | 50 | | |
54 | 51 | | |
55 | 52 | | |
| |||
85 | 82 | | |
86 | 83 | | |
87 | 84 | | |
88 | | - | |
89 | 85 | | |
90 | 86 | | |
91 | 87 | | |
| |||
0 commit comments