Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mise à disposition du convertisseur GTFS vers NeTEx #2049

Merged
merged 3 commits into from
Jan 17, 2022
Merged

Conversation

thbar
Copy link
Contributor

@thbar thbar commented Jan 13, 2022

Cette PR rend disponible le binaire de conversion GTFS vers NeTEx. C'est le bout de la chaîne de ces PR:

À considérer comme beta, car:

Reste à faire

  • Un petit test de conversion pour vérifier le binaire (avec un tout petit GTFS) serait bien (Tignes fait 80k et est rapide à convertir ; gtfs2netexfr --input ~/git/transport/tignes.zip --output tignes --participant test) -> à faire dans la vraie PR d'intégration plutôt
  • Vérification du test Dockerfile et correctif (il devrait initialement planter)
  • Review
  • Alerter @fchabouis -> précautions sur les jobs car il va falloir une queue de traitement spécifique, avec un max de 1, et on va sûrement avoir des délais de traitement très longs, et des problèmes potentiellement qui pourraient nous obliger à aller sur une autre taille de machine, ou un autre hébergeur juste pour cette partie, à voir !
  • Test sur prochainement ?

@thbar thbar changed the title Changement image de base (pour disposer du convertisseur GTFS -> NeTEx) Mise à disposition du convertisseur GTFS vers NeTEx Jan 13, 2022
@thbar thbar marked this pull request as ready for review January 13, 2022 16:04
@thbar thbar requested a review from a team as a code owner January 13, 2022 16:04
Copy link
Contributor

@fchabouis fchabouis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super, merci @thbar pour le gros boulot !

@thbar
Copy link
Contributor Author

thbar commented Jan 17, 2022

Testé sur prochainement, ça a bien booté, j'ai lancé un petit import et validation, ça n'a pas l'air de planter.

Je merge comme ça @fchabouis peut poursuivre derrière.

@thbar thbar merged commit c71348d into master Jan 17, 2022
@thbar thbar deleted the add-gtfs2netexfr branch January 17, 2022 14:04
@AntoineAugusti
Copy link
Member

Canon ✨

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants