Skip to content

Commit c81aaba

Browse files
DocGarbanzoclaude
andcommitted
Fix KerasLSTM xfail strictness and add .venv to .gitignore
CudnnRNNV3 is unsupported in TFLite on macOS but works on Linux; using strict=False allows XPASS on Linux and XFAIL on macOS without failing CI. Also add .venv to .gitignore to prevent accidental project-local venv commits. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 977839e commit c81aaba

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11

22
env/*
3+
.venv
34
data/*
45
dist/*
56

donkeycar/tests/test_keras.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,8 @@ def tmp_dir() -> str:
2626
KerasLinear, KerasCategorical, KerasInferred,
2727
# KerasLSTM uses CudnnRNNV3 which TFLite cannot run without CUDA
2828
pytest.param(KerasLSTM, marks=pytest.mark.xfail(
29-
reason='KerasLSTM uses CudnnRNNV3, not supported in TFLite',
30-
strict=True)),
29+
reason='KerasLSTM uses CudnnRNNV3, not supported in TFLite on macOS',
30+
strict=False)),
3131
KerasLocalizer, KerasIMU,
3232
Keras3D_CNN,
3333
KerasMemory, KerasBehavioral,

0 commit comments

Comments
 (0)