Merge remote-tracking branch 'origin/dev' into test_sparse

Author: igiloh-pinecone

pinecone/core/grpc/index_grpc.py

@@ -345,19 +345,19 @@ def _dict_to_grpc_vector(item):
             sparse_values = None
             if 'sparse_values' in item:
                 if not isinstance(item['sparse_values'], Mapping):
-                    raise ValueError(f"Column `sparse_values` is expected to be a dictionary, found {type(item['sparse_values'])}")
+                    raise TypeError(f"Column `sparse_values` is expected to be a dictionary, found {type(item['sparse_values'])}")
                 indices = item['sparse_values'].get('indices', None)
                 values = item['sparse_values'].get('values', None)
                 try:
                     sparse_values = GRPCSparseValues(indices=indices, values=values)
                 except TypeError as e:
-                    raise ValueError("Found unexpected data in column `sparse_values`. "
+                    raise TypeError("Found unexpected data in column `sparse_values`. "
                                      "Expected format is `'sparse_values': {'indices': List[int], 'values': List[float]}`."
                                      ) from e
 
-                metadata = item.get('metadata', None)
-                if metadata is not None and not isinstance(metadata, Mapping):
-                    raise TypeError(f"Column `metadata` is expected to be a dictionary, found {type(metadata)}")
+            metadata = item.get('metadata', None)
+            if metadata is not None and not isinstance(metadata, Mapping):
+                raise TypeError(f"Column `metadata` is expected to be a dictionary, found {type(metadata)}")
 
             try:
                 return GRPCVector(id=item['id'], values=item['values'], sparse_values=sparse_values,

pinecone/index.py

@@ -190,9 +190,9 @@ def _dict_to_vector(item):
                                      "Expected format is `'sparse_values': {'indices': List[int], 'values': List[float]}`."
                                      ) from e
 
-                metadata = item.get('metadata') or {}
-                if not isinstance(metadata, Mapping):
-                    raise TypeError(f"Column `metadata` is expected to be a dictionary, found {type(metadata)}")
+            metadata = item.get('metadata') or {}
+            if not isinstance(metadata, Mapping):
+                raise TypeError(f"Column `metadata` is expected to be a dictionary, found {type(metadata)}")
 
             try:
                 return Vector(id=item['id'], values=item['values'], sparse_values=sparse_values, metadata=metadata)