Skip to content

fix CI dependency bug #19

Closed
oliviamiller wants to merge 7 commits into
viam-modules:mainfrom
oliviamiller:bug-deps
Closed

fix CI dependency bug #19
oliviamiller wants to merge 7 commits into
viam-modules:mainfrom
oliviamiller:bug-deps

Conversation

@oliviamiller
Copy link
Copy Markdown
Collaborator

@oliviamiller oliviamiller commented Mar 23, 2026

Summary

This PR fixes a dependency conflict and CI build issue introduced by the tflite support PR.

After merging tflite support, the main branch CI failed because:

  • numpy==1.26.4 (required for tflite-runtime 2.14.0 compatibility) conflicted with newer versions of scipy and onnxruntime that the CI pip resolved to on a fresh install
  • The PR CI appeared to pass because viamrobotics/build-action was building the base branch (main) rather than the PR branch code — so dep changes were never actually tested in CI

Changes:

  • pin openwakeword transitive dependencies to prevent CI from picking up an incompatable version
  • build.yml — pass correct to the build action so the Viam cloud build actually tests PR branch code instead of always building main

Review
1 Approval needed

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.

1 participant