Fetch the model CI has been building without
Both failing jobs failed for the same reason, and it was not the reason either of them appeared to fail. Desktop reported a Clippy failure and Android reported the API-level check; both were `dr-segment`'s build script panicking because `models/yolo26n-seg.onnx` was a 133-byte LFS pointer rather than the 11 MB model. No checkout step asked for LFS objects. `.gitattributes` predicted this exactly — "a clone without git-lfs gets a ~130-byte pointer file where the model should be" — and the build script fails loudly by design rather than embedding a pointer and failing at inference. That design worked; nothing was reading the message. It stayed hidden because the Android job built `-p dr-gpu`, which never reaches dr-segment. Building the app crate does, which is how one fix surfaced another. Only the two jobs that compile get `lfs: true`. `layering` runs `cargo tree` and builds nothing, so it has no reason to fetch 11 MB. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -29,7 +29,18 @@ jobs:
|
||||
|
||||
steps:
|
||||
- name: Checkout
|
||||
# `lfs: true` is load-bearing, not hygiene. `core/dr-segment/models/*.onnx`
|
||||
# is tracked in LFS (see .gitattributes), and a checkout without it writes
|
||||
# a ~130-byte pointer where the 11 MB model should be. `dr-segment`'s build
|
||||
# script detects that and panics by design, so every job that compiles it
|
||||
# fails at the build script with a message about git-lfs — which reads like
|
||||
# a broken build rather than a missing fetch.
|
||||
#
|
||||
# It stayed hidden while the Android job built `-p dr-gpu`, which does not
|
||||
# reach dr-segment. Building the app crate does.
|
||||
uses: actions/checkout@v4
|
||||
with:
|
||||
lfs: true
|
||||
|
||||
- name: Cache cargo
|
||||
uses: actions/cache@v4
|
||||
@@ -84,7 +95,18 @@ jobs:
|
||||
|
||||
steps:
|
||||
- name: Checkout
|
||||
# `lfs: true` is load-bearing, not hygiene. `core/dr-segment/models/*.onnx`
|
||||
# is tracked in LFS (see .gitattributes), and a checkout without it writes
|
||||
# a ~130-byte pointer where the 11 MB model should be. `dr-segment`'s build
|
||||
# script detects that and panics by design, so every job that compiles it
|
||||
# fails at the build script with a message about git-lfs — which reads like
|
||||
# a broken build rather than a missing fetch.
|
||||
#
|
||||
# It stayed hidden while the Android job built `-p dr-gpu`, which does not
|
||||
# reach dr-segment. Building the app crate does.
|
||||
uses: actions/checkout@v4
|
||||
with:
|
||||
lfs: true
|
||||
|
||||
- name: Cache cargo
|
||||
uses: actions/cache@v4
|
||||
|
||||
Reference in New Issue
Block a user