Merge branch 'master' into qa

Author: naquib314

breadbox/breadbox/api/datasets.py

@@ -160,9 +160,7 @@ def get_feature_data(
 ):
     """
     Load data for each of the given dataset_id, feature_id pairs.
-    This differs from the /get-features endpoint in the type of ID it
-    accepts as input and the format of the response. This endpoint also
-    does not do any filtering or grouping of feature values.
+    This endpoint does not do any filtering or grouping of feature values.
     """
     if len(dataset_ids) != len(feature_ids):
         raise UserError(

…lorer_2/context/test_custom_operators.py …sts/depmap_compute_embed/test_context.pyportal-backend/tests/depmap/data_explorer_2/context/test_custom_operators.py renamed to breadbox/tests/depmap_compute_embed/test_context.py portal-backend/tests/depmap/data_explorer_2/context/test_custom_operators.py renamed to breadbox/tests/depmap_compute_embed/test_context.py

@@ -1,5 +1,7 @@
-from depmap_compute.context import LegacyContextEvaluator
+from breadbox.depmap_compute_embed.context import ContextEvaluator
 
+# Our ContextEvaluator makes heavy use of an extension to the 3rd party library: json_logic
+# These tests ensure that behavior is continuing to work as expected. 
 
 def test_operator__not_in():
     assert expressions_are_equivalent(
@@ -76,6 +78,6 @@ def expressions_are_equivalent(boolean_value, json_logic_expr):
     # These expressions don't use variables (just pure logic)
     var_name = "dummy variable"
     get_slice_data_mock = lambda _: {}
-    result = LegacyContextEvaluator(context, get_slice_data_mock).is_match(var_name)
+    result = ContextEvaluator(context, get_slice_data_mock).is_match(var_name) # pyright: ignore
 
-    return result == boolean_value
+    return result == boolean_value