Introduce the implementation of population sampling within the package

mobility-team / mobility

Mobility, an open-source library for mobility modelisation

MIT License

16 stars 10 forks source link

Introduce the implementation of population sampling within the package #89

Open lucas-boh opened 7 months ago

lucas-boh commented 7 months ago

J'ai commencé à intégrer le module population à mobility. Le script pour l'échantillonnage reste le même pour l'instant, mais le téléchargement des données nécessaires est automatisé (comme dans _get_inseedata.py), avec un dépôt des données sur le data.gouv de mobility.

Les changements principaux :

les données des ménages/ logements sont récupérées et préparées par "prepare_dwellings.py"
les données du découpage administratif sont récupérées et préparées par "prepare_cities.py"
l'exemple de calcul de l'empreinte carbone est mis à jour pour fonctionner avec l'intégration du module d'échantillonnage.

À poursuivre :

Documenter et intégrer davantage le code de l'échantillonnage de population.
Vérifier que l'échantillonnage donne des valeurs cohérentes, notamment pour le calcul des empreintes carbone.

codecov[bot] commented 7 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (9dfad7a) 95.00% compared to head (21fb930) 95.00%. Report is 2 commits behind head on main.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #89 +/- ## ======================================= Coverage 95.00% 95.00% ======================================= Files 19 19 Lines 961 961 ======================================= Hits 913 913 Misses 48 48 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Mind-the-Cap commented 6 months ago

Salut @lucas-boh, merci pour ce travail !

Population a l'air d'être sous une licence Apache 2, qui est compatible avec la licence MIT de mobility, mais qui nécessite d'être conservée pour le code réutilisé.

Plusieurs solutions :

indiquer plusieurs licences pour le projet
l'indiquer spécifiquement dans ce dossier, et mettre une phrase ailleurs dans la présentation globale du package. @FlxPo preneuse de ton opinion

FlxPo commented 6 months ago

@lucas-boh à quel point ton code est une réécriture du code d'elioth ? On pourrait demander à Elioth s'ils sont OK pour une réutilisation de leur code sans modification de licence ?

(juste pour simplifier le travail)

lucas-boh commented 6 months ago

Pour l'instant j'ai simplement changé quelques lignes dans leur script, mais globalement c'est pareil pour l'échantillonnage. La préparation des données est originale en revanche.