[Backport release/0.0.14] build: fetch LFS test data before the suites run - #4181
github-actions[bot] wants to merge 1 commit into
Conversation
(cherry picked from commit 485c8c8)
|
| if grep -rqF -e "\"$n\"" -e "'$n'" --include='*.py' dimos; then | ||
| include+=("$f") | ||
| fi |
There was a problem hiding this comment.
The selector only includes an archive when its complete filename appears as a quoted Python literal. The occupancy tests construct filenames from palette or method, so their existing archives—including visualize_occupancy_rainbow.png, resample_path_simple.png, and gradient_voronoi.png—are not prefetched. When those files are absent locally, get_data() still downloads them during the test, leaving the download-delay timeout this change is intended to avoid.
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Artifacts
Dynamic asset selection validation script
- The executable script reproduces the changed archive-selection predicate against dynamically generated test-data names.
Dynamic asset selection validation output
- The captured output shows that all six existing dynamic asset archives are excluded by the selector.
Description
Backport of #4106 to
release/0.0.14.