From a0c050baac91e4fd65115e47d1078dc3a55eb0cb Mon Sep 17 00:00:00 2001 From: Jakob Schnell Date: Wed, 15 May 2024 15:30:43 +0200 Subject: [PATCH] docs: add TODOs --- ORStools/common/directions_core.py | 2 ++ ORStools/proc/directions_points_layer_proc.py | 1 + 2 files changed, 3 insertions(+) diff --git a/ORStools/common/directions_core.py b/ORStools/common/directions_core.py index 9d960f1f..a48116dc 100644 --- a/ORStools/common/directions_core.py +++ b/ORStools/common/directions_core.py @@ -262,9 +262,11 @@ def get_extra_info_features_directions(response: dict, extra_info_order: list[st values = extra_info[key]["values"] for val in values: for i in range(val[0], val[1]): + # TODO: translate integer val[2] to readable value according to BE docs extras_list[key].append(val[2]) for i in range(len(coordinates) - 1): + # TODO: dissolve, but is one layer per extra info more valuable? feat = QgsFeature() qgis_coords = [QgsPoint(x, y, z) for x, y, z in coordinates[i : i + 2]] feat.setGeometry(QgsGeometry.fromPolyline(qgis_coords)) diff --git a/ORStools/proc/directions_points_layer_proc.py b/ORStools/proc/directions_points_layer_proc.py index dd479e05..95757ea6 100644 --- a/ORStools/proc/directions_points_layer_proc.py +++ b/ORStools/proc/directions_points_layer_proc.py @@ -99,6 +99,7 @@ def __init__(self): self.EXTRA_INFO, self.tr("Extra Info"), options=EXTRA_INFOS, + # TODO: allowMultiple=False? This way, we wouldn't have to worry about dissolving more stuff. allowMultiple=True, optional=True, ),