Enable python nodes and ovms, geti calculators in new python lib#134
Open
rasapala wants to merge 2 commits into
Open
Enable python nodes and ovms, geti calculators in new python lib#134rasapala wants to merge 2 commits into
rasapala wants to merge 2 commits into
Conversation
…library solution for mediapipe.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR aligns the MediaPipe fork with OVMS’s new Python runtime-shared architecture so CLIP Python-node graphs can load and run reliably.
Why this was needed:
OVMS moved Python calculators to dynamic/runtime-shared loading, but MediaPipe Geti/OVMS calculator deps still assumed the old in-tree/static wiring.
That caused build mismatches and runtime registration/load issues (including stale targets and duplicated proto ownership).
What was changed:
Switched Geti inference to use OVMS KServe proto (@ovms//src/kfserving_api:kfserving_api_cpp) instead of local grpc_predict_v2_* targets.
Added runtime-focused OVMS calculator targets (ovms_calculator_runtime, openvinoinferencecalculator_runtime) to support one-point calculator registration in the runtime-shared flow.
Updated OpenVINOInferenceCalculator to support runtime mode with TensorFlow/TFLite paths compile-disabled via OVMS_RUNTIME_DISABLE_TF_TENSORS and explicit guards for unsupported TF/TFLite packet tags in that mode.
Follow-up build fix removed stale grpc_predict_v2_* references from //mediapipe:geti_targets after proto ownership moved to OVMS.
Outcome:
Restores successful MediaPipe fork build in the new integration model.
Unblocks CLIP Python nodes in OVMS runtime-shared setup.
Keeps calculator registration and dependency ownership consistent between MediaPipe and OVMS.