Change default temporal property to datetime

improve command typing
columns.value should allow list and tuple

Author: ivorbosloper

CHANGELOG.md

@@ -7,6 +7,9 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ## [Unreleased]
 
+- Change default temporal property to datetime
+- Enable Converter.columns list and tuple types
+
 ## [v0.2.11] - 2025-10-09
 
 - XML/HTML-like tags (with < and > characters) in logs are properly escaped for loguru

vecorel_cli/conversion/base.py

@@ -11,7 +11,7 @@
 from glob import glob
 from io import StringIO
 from tempfile import TemporaryDirectory
-from typing import Any, Callable, Generator, Optional
+from typing import Any, Callable, Generator, Optional, Sequence
 
 import flatdict
 import geopandas as gpd
@@ -51,7 +51,7 @@ class BaseConverter(LoggerMixin):
     variants: dict[str, Sources] = {}
     variant: Optional[str] = None
 
-    columns: dict[str, str] = {}
+    columns: dict[str, str|Sequence[str]] = {}
     column_additions: dict[str, str] = {}
     column_filters: dict[str, Callable] = {}
     column_migrations: dict[str, Callable] = {}
@@ -325,7 +325,6 @@ def convert(
         paths = self.download_files(urls, cache, **request_args)
 
         gdf = self.read_data(paths, **self.open_options)
-
         self.info("GeoDataFrame created from source(s):")
         # Make it so that everything is shown, don't output ... if there are too many columns or rows
         display_pandas_unrestricted()
@@ -375,7 +374,7 @@ def convert(
         for old_key, new_key in columns.items():
             if old_key in gdf.columns:
                 # If the new keys are a list, duplicate the column
-                if isinstance(new_key, list):
+                if isinstance(new_key, (list, tuple)):
                     for key in new_key:
                         gdf[key] = gdf.loc[:, old_key]
                         actual_columns[key] = key
@@ -417,6 +416,7 @@ def convert(
         columns = list(actual_columns.values())
         pq = GeoParquet(output_file)
         pq.set_collection(self.create_collection(cid))
+
         pq.write(
             gdf,
             properties=columns,

vecorel_cli/convert.py

@@ -74,7 +74,7 @@ def get_cli_args():
                 "-og",
                 is_flag=True,
                 type=click.BOOL,
-                help="Keep the source geometries as provided, i.e. this option disables that geomtries are made valid and converted to Polygons.",
+                help="Keep the source geometries as provided, i.e. this option disables that geometries are made valid and converted to Polygons.",
                 default=False,
             ),
             "py-package": PY_PACKAGE,

vecorel_cli/create_stac.py

@@ -25,7 +25,7 @@ class CreateStacCollection(BaseCommand):
     processing_extension = "https://stac-extensions.github.io/processing/v1.1.0/schema.json"
     table_extension = "https://stac-extensions.github.io/table/v1.2.0/schema.json"
 
-    temporal_property = "determination_datetime"
+    temporal_property = "datetime"
 
     @staticmethod
     def get_cli_args():

vecorel_cli/improve.py

@@ -78,7 +78,6 @@ def improve_file(
         input_encoding = create_encoding(source)
         geodata = input_encoding.read()
         collection = input_encoding.get_collection()
-
         geodata, collection = self.improve(geodata, collection=collection, **kwargs)
 
         output_encoding = create_encoding(target)
@@ -94,8 +93,8 @@ def improve_file(
     def improve(
         self,
         gdf: GeoDataFrame,
-        collection: dict = {},
-        rename: dict[str, str] = {},
+        collection: Collection,
+        rename: Optional[dict[str, str]] = None,
         add_sizes: bool = False,
         fix_geometries: bool = False,
         explode_geometries: bool = False,
@@ -117,7 +116,7 @@ def improve(
             self.info("Exploded geometries")
 
         # Rename columns
-        if len(rename) > 0:
+        if rename:
             self.rename_warnings(gdf, rename)
             gdf, collection = self.rename_properties(gdf, rename, collection)
             self.info("Renamed columns")