-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #2 from VincentHardouin/use-gtfs-2
[FEATURE] Utiliser les données provenant du GTFS
- Loading branch information
Showing
16 changed files
with
54,366 additions
and
56,317 deletions.
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
# 2024-04-14 | ||
|
||
J'aimerais merger la PR aujourd'hui sur l'import et l'export des données provenant de GTFS. | ||
J'ai des tests liés à mon environnement (charge des fichiers localement) que je dois modifier ou supprimer, car je ne | ||
veux pas publier les données d'IDFM. | ||
|
||
J'ai aussi envie d'harmoniser les noms, il y a des mélanges entre `stops` et `stations`. Je ne suis pas tout à fait au | ||
clair sur le nommage. | ||
|
||
Bon, j'ai mergé sans harmoniser finalement, je le ferai plus tard. J'ai aussi skipé les tests qui ne sont pas jouables | ||
sans les données. | ||
J'ai d'ailleurs eu des erreurs de flaky tests, que j'ai pu corriger en ajoutant l'option `--no-file-parallelism` | ||
|
||
Maintenant, il est temps d'utiliser les nouvelles données, je migre la class Graph, puis pick-station. Il reste plus que | ||
tout ce qui est affichage à migrer. |
Oops, something went wrong.