From b77f73b418dd716ce9a7569120de197467641b79 Mon Sep 17 00:00:00 2001 From: Hammad Bashir Date: Tue, 17 Sep 2024 13:47:19 -0700 Subject: [PATCH] [BUG] Fix CVP tests by forcing numpy reimport (#2814) ## Description of changes *Summarize the changes made by this PR.* - Improvements & Bug fixes - Fix CVP tests which broke in #2811 by reimporting numpy - New functionality - None ## Test plan *How are these changes tested?* - [x] Tests pass locally with `pytest` for python, `yarn test` for js, `cargo test` for rust ## Documentation Changes None --- chromadb/test/property/test_cross_version_persist.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/chromadb/test/property/test_cross_version_persist.py b/chromadb/test/property/test_cross_version_persist.py index 91ea7bb76c1..f846f95ec02 100644 --- a/chromadb/test/property/test_cross_version_persist.py +++ b/chromadb/test/property/test_cross_version_persist.py @@ -35,7 +35,7 @@ version_re = re.compile(r"^[0-9]+\.[0-9]+\.[0-9]+$") # Some modules do not work across versions, since we upgrade our support for them, and should be explicitly reimported in the subprocess -VERSIONED_MODULES = ["pydantic"] +VERSIONED_MODULES = ["pydantic", "numpy"] def versions() -> List[str]: