IGNF / myria3d

Myria3D: Aerial Lidar HD Semantic Segmentation with Deep Learning
https://ignf.github.io/myria3d/
BSD 3-Clause "New" or "Revised" License
170 stars 23 forks source link

load_las est une classmethod - risque d'erreur si accès concurrent ? #19

Closed CharlesGaydon closed 2 years ago

CharlesGaydon commented 2 years ago

Par contre le classmethod semble nécessaire pour pouvoir spécifier directement la fonction dans le paramétrage hydra qui indique la fonction de chargement. Ca se passe dans la description de dataset :

# loading functions for data object prepared via lidar_multiclass/datamodule/data.py
load_las_func:
  _target_: functools.partial
  _args_:
    - "${get_method:lidar_multiclass.data.loading.FrenchLidarDataLogic.load_las}"

On pourrait envisager de remplace par une fonction directement mais ça oblige à créer une fonction avec une signature à rallonge. Effectivement le risque de concurrence d'accès existe, même si il me semble faible ; la classe n'est pas modifiée par la méthode.

_Originally posted by @CharlesGaydon in https://github.com/IGNF/lidar-deep-segmentation/pull/14#discussion_r873899531_

CharlesGaydon commented 2 years ago

On a éliminé cette logique dans https://github.com/IGNF/myria3d/pull/33.