Move concept fields to child objects

This led to confusion when interacting with the hits, in that passages
seemed to have `concept_counts` and documents, `concepts`. Moving both
to just be on the child object instead makes it clearer the distinction
between them

src/cpr_sdk/models/search.py

@@ -437,10 +437,8 @@ class Hit(BaseModel):
     corpus_type_name: Optional[str] = None
     corpus_import_id: Optional[str] = None
     metadata: Optional[Sequence[dict[str, str]]] = None
-    concepts: Optional[Sequence[Concept]] = None
     relevance: Optional[float] = None
     rank_features: Optional[dict[str, float]] = None
-    concept_counts: Optional[dict[str, int]] = None
 
     @classmethod
     def from_vespa_response(cls, response_hit: dict) -> "Hit":
@@ -485,6 +483,8 @@ def __eq__(self, other):
 class Document(Hit):
     """A document search result hit."""
 
+    concept_counts: Optional[dict[str, int]] = None
+
     @classmethod
     def from_vespa_response(cls, response_hit: dict) -> "Document":
         """
@@ -519,7 +519,6 @@ def from_vespa_response(cls, response_hit: dict) -> "Document":
             corpus_type_name=fields.get("corpus_type_name"),
             corpus_import_id=fields.get("corpus_import_id"),
             metadata=fields.get("metadata"),
-            concepts=fields.get("concepts"),
             relevance=response_hit.get("relevance"),
             rank_features=fields.get("summaryfeatures"),
             concept_counts=fields.get("concept_counts"),
@@ -534,6 +533,7 @@ class Passage(Hit):
     text_block_type: str
     text_block_page: Optional[int] = None
     text_block_coords: Optional[Sequence[tuple[float, float]]] = None
+    concepts: Optional[Sequence[Concept]] = None
 
     @classmethod
     def from_vespa_response(cls, response_hit: dict) -> "Passage":

src/cpr_sdk/version.py

@@ -1,6 +1,6 @@
 _MAJOR = "1"
 _MINOR = "16"
-_PATCH = "1"
+_PATCH = "2"
 _SUFFIX = ""
 
 VERSION_SHORT = "{0}.{1}".format(_MAJOR, _MINOR)