Merge pull request #147 from 3DBAG/fix/floors-estimation-test

Fix/floors estimation test

Author: balazsdukai

README.md

@@ -68,8 +68,7 @@ at your option.
 ## Contribution
 
 Unless you explicitly state otherwise, any contribution intentionally submitted
-for inclusion in the work by you, as defined in the Apache-2.0 license, shall be dual licensed as above, without any
-additional terms or conditions.
+for inclusion in the work by you, as defined in the Apache-2.0 license, shall be dual licensed as above, without any additional terms or conditions.
 
 ## 3DBAG organisation
 

docker/postgres/Dockerfile

@@ -1,4 +1,4 @@
-FROM postgis/postgis:14-3.4 
+FROM postgis/postgis:14-3.4
 
 LABEL org.opencontainers.image.authors="Gina Stavropoulou <[email protected]>, Balázs Dukai <[email protected]>"
 

makefile

@@ -88,7 +88,7 @@ include .env
 download:
 	rm -rf $(BAG3D_TEST_DATA)
 	mkdir -p $(BAG3D_TEST_DATA)
-	cd $(BAG3D_TEST_DATA) ; curl -O https://data.3dbag.nl/testdata/pipeline/test_data_v7.zip ; unzip -q test_data_v7.zip ; rm test_data_v7.zip
+	cd $(BAG3D_TEST_DATA) ; curl -O https://data.3dbag.nl/testdata/pipeline/test_data_v8.zip ; unzip -q test_data_v8.zip ; rm test_data_v8.zip
 
 
 install_uv:

packages/floors_estimation/tests/test_floors_estimation.py

@@ -1,6 +1,5 @@
 from pathlib import Path
 
-import pytest
 from bag3d.common.types import PostgresTableIdentifier
 from bag3d.common.utils.database import table_exists
 from bag3d.floors_estimation.assets.floors_estimation import (
@@ -97,7 +96,6 @@ def test_all_features(context):
     assert table_exists(context, all_features_table) is True
 
 
-@pytest.mark.skip(reason="Needs fixing")
 def test_preprocessed_features(context):
     all_features_table = PostgresTableIdentifier(
         "floors_estimation", "building_features_all"