search

mobility-team / mobility

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

Compute the internal distance within the code instead of using a CSV #43

Closed Nouhaila13 closed 1 year ago

Nouhaila13 commented 1 year ago

Compute the internal distance within the code instead of using a CSV and Generalise the code, so it can be used for any set of contiguous departments.

Mind-the-Cap commented 1 year ago

Bonjour @Nouhaila13, merci beaucoup !

Comme vous pouvez le voir dans la liste des commits et dans Files changed, il y a beaucoup de modifications qui ne relèvent pas de cette PR. C'est probablement que vous êtes parti·es d'une ancienne version du code. Pouvez-vous rebase la PR s'il vous plaît (c'est peut-être possible avec Github Desktop, il est également possible de refaire les modifications à la main depuis main s'il y en a peu) ? Ainsi je pourrai la relire et l'approuver.

Nouhaila13 commented 1 year ago

Bonjour madame @Mind-the-Cap , Je tiens à vous assurer que nous avons travaillé sur la dernière version du code. Cordialement.

Mind-the-Cap commented 1 year ago

@Nouhaila13 malheureusement vous pouvez constater que les changements sont un peu trop nombreux : https://github.com/mobility-team/mobility/pull/43/files Cela m'était beaucoup arrivé à mes débuts ! Je sais que c'est un peu décourageant mais pouvez-vous tenter d'appliquer l'une des méthodes que je vous ai proposées s'il vous plaît ? Merci par avance !