add comment

openeo_processes_dask/process_implementations/cubes/merge.py

@@ -113,6 +113,8 @@ def merge_cubes(
                     cube1 = cube1.to_dataset(cube1.openeo.band_dims[0])
                     cube2 = cube2.to_dataset(cube2.openeo.band_dims[0])
 
+                # compat="override" to deal with potentially conflicting coords
+                # see https://github.com/Open-EO/openeo-processes-dask/pull/148 for context
                 merged_cube = xr.combine_by_coords(
                     [cube1, cube2], combine_attrs="drop_conflicts", compat="override"
                 )

tests/test_merge.py

@@ -149,9 +149,10 @@ def test_merge_cubes_type_4(
 
 @pytest.mark.parametrize("size", [(6, 5, 4, 1)])
 @pytest.mark.parametrize("dtype", [np.float64])
-def test_merge_cubes_type_5(
+def test_conflicting_coords(
     temporal_interval, bounding_box, random_raster_data, process_registry
 ):
+    # See https://github.com/Open-EO/openeo-processes-dask/pull/148 for why is is necessary
     # This is basically broadcasting the smaller datacube and then applying the overlap resolver.
     cube_1 = create_fake_rastercube(
         data=random_raster_data,