Merge branch 'main' into issues/1543-none-epsg

Author: gadomski

CHANGELOG.md

@@ -9,6 +9,7 @@
 ### Fixed
 
 - `proj:epsg` migration when `None` ([#1544](https://github.com/stac-utils/pystac/pull/1544))
+- fixed missing parameter "title" in pystac.extensions.classification.Classification ([#1539](https://github.com/stac-utils/pystac/pull/1539))
 
 ## [v1.12.2]
 

pystac/extensions/classification.py

@@ -60,6 +60,7 @@ def apply(
         value: int,
         description: str | None = None,
         name: str | None = None,
+        title: str | None = None,
         color_hint: str | None = None,
         nodata: bool | None = None,
         percentage: float | None = None,
@@ -74,6 +75,7 @@ def apply(
             name: Short name of the class for machine readability. Must consist only
                 of letters, numbers, -, and _ characters. Required as of v2.0 of
                 this extension.
+            title: Human-readable name for use in, e.g., a map legend
             color_hint: An optional hexadecimal string-encoded representation of the
                 RGB color that is suggested to represent this class (six hexadecimal
                 characters, all capitalized)
@@ -90,6 +92,7 @@ def apply(
                 "As of v2.0.0 of the classification extension, 'name' is required"
             )
         self.name = name
+        self.title = title
         self.description = description
         self.color_hint = color_hint
         self.nodata = nodata
@@ -108,6 +111,7 @@ def create(
         value: int,
         description: str | None = None,
         name: str | None = None,
+        title: str | None = None,
         color_hint: str | None = None,
         nodata: bool | None = None,
         percentage: float | None = None,
@@ -122,6 +126,7 @@ def create(
             name: Short name of the class for machine readability. Must consist only
                 of letters, numbers, -, and _ characters. Required as of v2.0 of
                 this extension.
+            title: Human-readable name for use in, e.g., a map legend
             color_hint: An optional hexadecimal string-encoded representation of the
                 RGB color that is suggested to represent this class (six hexadecimal
                 characters, all capitalized)
@@ -134,6 +139,7 @@ def create(
         c.apply(
             value=value,
             name=name,
+            title=title,
             description=description,
             color_hint=color_hint,
             nodata=nodata,
@@ -189,6 +195,17 @@ def name(self, v: str) -> None:
             )
         self.properties["name"] = v
 
+    @property
+    def title(self) -> str | None:
+        return self.properties.get("title")
+
+    @title.setter
+    def title(self, v: str) -> None:
+        if v is not None:
+            self.properties["title"] = v
+        else:
+            self.properties.pop("title", None)
+
     @property
     def color_hint(self) -> str | None:
         """Get or set the optional color hint for this class.

tests/extensions/cassettes/test_view/ViewTest.test_azimuth.yaml renamed to tests/extensions/cassettes/test_view/test_azimuth.yaml

@@ -60,12 +60,23 @@ def test_stac_extensions(landsat_item: Item) -> None:
 
 def test_classification_object() -> None:
     c = Classification.create(
-        name="dummy", description="empty class", value=0, color_hint="FF00AB"
+        name="dummy",
+        description="empty class",
+        value=0,
+        title="dummy title",
+        color_hint="FF00AB",
+        nodata=True,
+        percentage=20.3,
+        count=2,
     )
     assert c.name == "dummy"
     assert c.description == "empty class"
     assert c.color_hint == "FF00AB"
     assert c.value == 0
+    assert c.title == "dummy title"
+    assert c.nodata is True
+    assert c.percentage == 20.3
+    assert c.count == 2
 
     assert Classification(c.to_dict()) == c
     with pytest.raises(NotImplementedError):