Skip to content

Commit

Permalink
changes according to Palvina's code review
Browse files Browse the repository at this point in the history
  • Loading branch information
MichelDaab committed Aug 5, 2024
1 parent 5677001 commit 5d7230f
Show file tree
Hide file tree
Showing 8 changed files with 31 additions and 34 deletions.
25 changes: 25 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -108,3 +108,28 @@ Pour lancer les tests :
```
python -m pytest -s
```
### paramètres pour créer les squelettes des cours d'eau
Pour fonctionner, la création de squelettes a besoin d'une série de paramètres, certains ayant une valeur par défaut, d'autres non. Les paramètres se trouvent dans le fichier configs/configs_lidro.yaml. On peut soit les y modifier, soit les modifer en ligne de commande lors de l'exécution du script avec :
```
python lidro/main_create_skeleton_lines.py [nom_paramètre_1]=[valeur_du_paramètre_1] [nom_paramètre_2]=[valeur_du_paramètre_2]
```
ces paramètres sont :
SKELETON.FILE_PATH.MASK_INPUT_PATH : Le chemin d'entrée des masques des cours d'eau
SKELETON.FILE_PATH.SKELETON_LINES_OUTPUT_PATH : Le chemin de sortie des squelettes uniquement (pas de fichier de sortie si laissé à vide)
SKELETON.FILE_PATH.GAP_LINES_OUTPUT_PATH : Le chemin de sortie des lignes franchissant des ponts uniquement (pas de fichier de sortie si laissé à vide)
SKELETON.FILE_PATH.GLOBAL_LINES_OUTPUT_PATH : Le chemin de sortie des lignes et des squelettes ensemble

SKELETON.MAX_GAP_WIDTH : La distance maximale envisagée pour franchir des ponts
SKELETON.MAX_BRIDGES : Le nombre maximal de ponts entre deux bras séparés de cours d'eau
SKELETON.GAP_WIDTH_CHECK_DB : La distance à partir de laquelle on vérifie via la base de données s'il y a bien un pont
SKELETON.RATIO_GAP : la proportion de la ligne franchissant un pont qui est comparé en base pour voir s'il y a bien un pont (trop grande et on pourrait trouver un pont qui ne correspond pas)

SKELETON.DB_UNI.DB_NAME : Le nom de la base de données
SKELETON.DB_UNI.DB_HOST : l'adresse de la base de données
SKELETON.DB_UNI.DB_USER : L'utilisateur de la base de données
SKELETON.DB_UNI.DB_PASSWORD : Le mot de passe de l'utilisateur
SKELETON.DB_UNI.DB_PORT : La port de connexion avec la base de données

SKELETON.BRANCH.VORONOI_MAX_LENGTH : LA longuer maximum des lignes individuelles des squelettes
SKELETON.BRANCH.WATER_MIN_SIZE : La longueur minimal à partir de laquelle une ligne de squelette sera automatiquement gardée (trop petite, et il y aura des sortes "d'aiguilles" qui apparaitront. Trop grande, et certains afluents ne seront pas détectés)
SKELETON.BRANCH.MAX_GAP_CANDIDATES : Le nombre maximum de candidats pour envisager de franchir des ponts entre deux bras
4 changes: 2 additions & 2 deletions lidro/skeleton/group_maker.py
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
class GroupMaker:
"""
From a list of elements, GroupMaker is used to
group sets of elements according to single element from
the sets. For example:
put together groups of elements, according to single
element from each group. For example:
At the beginning, we have {A}, {B}, {C}, {D}, {E}
put_together(A, B) -> {A, B}, {C}, {D}, {E}
put_together(C, D) -> {A, B}, {C, D}, {E}
Expand Down
2 changes: 1 addition & 1 deletion test/skeleton/test_branch.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@

sys.path.append('lidro/skeleton')

BRANCH_TEST_1_PATH = "test_files/90.geojson"
BRANCH_TEST_1_PATH = "data/skeleton_hydro/test_files/90.geojson"
CRS_FOR_TEST = 2145
WATER_MIN_SIZE_TEST = 20

Expand Down
6 changes: 3 additions & 3 deletions test/skeleton/test_main_skeleton.py
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,9 @@
DB_UNI_USER = os.getenv('DB_UNI_USER')
DB_UNI_PASSWORD = os.getenv('DB_UNI_PASSWORD')
CRS = 2154
MAIN_SKELETON_TEST_1_1_PATH = "test_files/40.geojson"
MAIN_SKELETON_TEST_1_2_PATH = "test_files/43.geojson"
MAIN_SKELETON_TEST_1_3_PATH = "test_files/44.geojson"
MAIN_SKELETON_TEST_1_1_PATH = "data/skeleton_hydro/test_files/40.geojson"
MAIN_SKELETON_TEST_1_2_PATH = "data/skeleton_hydro/test_files/43.geojson"
MAIN_SKELETON_TEST_1_3_PATH = "data/skeleton_hydro/test_files/44.geojson"


def test_main_skeleton_1():
Expand Down
7 changes: 0 additions & 7 deletions test_files/40.geojson

This file was deleted.

7 changes: 0 additions & 7 deletions test_files/43.geojson

This file was deleted.

7 changes: 0 additions & 7 deletions test_files/44.geojson

This file was deleted.

Loading

0 comments on commit 5d7230f

Please sign in to comment.