mobility-team / mobility

Mobility, an open-source library for mobility modelisation
MIT License
16 stars 10 forks source link

[Projet] Définir une stratégie d'utilisation des fichiers externes #23

Closed Mind-the-Cap closed 1 year ago

Mind-the-Cap commented 1 year ago

Il faut que l'on décide d'une stratégie d'utilisation de fichiers externes dans notre code:

  • Inclure systématiquement les fichiers nécessaires pour faire tourner les fonctions et les exemples ?
  • Définir des jeux de données "de base" et d'autres qu'on laisse l'utilisateur gérer ?
  • Intégrer ou non la mise à jour des jeux de données (millésimes du recensement INSEE par exemple) ?

@FlxPo dans https://github.com/mobility-team/mobility/issues/22

Personnellement, je suis en faveur d'inclure systématiquement les fichiers nécessaires :

Je ne vois pas de cas pour l'instant où il serait intéressant que l'utilisateur ait des données à gérer. Je suis en faveur d'avoir les différentes versions, grâce à un dictionnaire qui enregistrerait les différents liens. Mais dans l'immédiat, on peut rester sur les dernières données, on n'a pas de gros cas d'usage pour ça (dans le futur je vois la reproductibilité notamment, ou la comparaison entre années).

FlxPo commented 1 year ago

D'accord avec ta règle, on fixe une limite de taille de fichier pour Github ? 1 Mo, 10 Mo par exemple ? Il faut aussi anticiper l'utilisation du package après une installation pip : bien définir les fichiers nécessaires au code et ceux qui sont utiles uniquement pour les exemples et tests.

Mind-the-Cap commented 1 year ago

Je dirais 50 Mo, qui est la limite d'avertissement de Github. J'imagine que les fichiers nécessaires au code doivent être dans mobility/data et ceux dans les exemples dans le répertoire dédié. Comme ça, à terme on pourra scinder si ça devient trop volumineux (avec un repo mobility-examples)

FlxPo commented 1 year ago

J'ajoute une stratégie supplémentaire :

Mind-the-Cap commented 1 year ago
Mind-the-Cap commented 1 year ago

Ajouté dans la documentation avec https://github.com/mobility-team/mobility/pull/65