joeyutong opened a new pull request, #626: URL: https://github.com/apache/flink-agents/pull/626
## Summary - replace the test-only mock embedding implementation in `test_vector_store_long_term_memory.py` - remove the dependency on ChromaDB `DefaultEmbeddingFunction()` for non-Ollama test runs - keep the existing add/get/search assertions unchanged with deterministic local embeddings ## Why The previous mock embedding path could trigger an implicit model download through ChromaDB even though this unit test only needs deterministic embeddings for local assertions. ## Validation - `/Users/tangyu/Workspaces/flink-agents/venv/bin/python -m pytest /tmp/flink-agents-chroma-test-no-download/python/flink_agents/runtime/memory/tests/test_vector_store_long_term_memory.py -q` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
