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

Le SIRI de Brest est labellisé SIRI Lite #4184

Open
thbar opened this issue Sep 12, 2024 · 7 comments
Open

Le SIRI de Brest est labellisé SIRI Lite #4184

thbar opened this issue Sep 12, 2024 · 7 comments
Assignees

Comments

@thbar
Copy link
Contributor

thbar commented Sep 12, 2024

GT7 aujourd'hui et on a parlé du requêteur SIRI du PAN, et un échange avec Emmanuelle Magnifici (RATP Dev) me fait me rendre compte que le flux de Brest semble mal configuré:

CleanShot 2024-09-12 at 15 07 13@2x

On ne voit pas du coup le lien "essayez moi" habituellement présent sur les flux SIRI, mais le serveur répond bien comme un serveur SIRI:

https://transport.data.gouv.fr/tools/siri-querier?endpoint_url=https%3A%2F%2Fara-api.enroute.mobi%2Fbibus%2Fsiri&requestor_ref=opendata&query_template=LinesDiscovery

CleanShot 2024-09-12 at 15 10 28@2x

Je crée le ticket et je le transmets directement à Emmanuelle qui est dans le chat du GT7 en ce moment.

@AntoineAugusti
Copy link
Member

AntoineAugusti commented Sep 12, 2024

La description de la ressource SIRI contient le texte suivant dans la description

SIRI Lite VehicleMonitoring

Ceci fait que la ressource est détectée comme SIRI Lite. Supprimer ceci devrait corriger le problème.

def siri_lite?(%{} = params) do
cond do
ods_resource?(params) or documentation?(params) -> false
format?(params, "SIRI Lite") -> true
siri_lite?(params["title"]) -> true
siri_lite?(params["description"]) -> true
siri_lite?(params["url"]) -> true
true -> false
end
end

On détecte le SIRI Lite avant le SIRI (car SIRI est "contenu" dans SIRI Lite).

@thbar
Copy link
Contributor Author

thbar commented Sep 12, 2024

Emmanuelle m'indique qu'ils ont à la fois du SIRI et du SIRI Lite. Ceci explique peut-être cela.

Sans faire de "y'a qu'à faut qu'on", il sera peu fiable d'utiliser la description pour ça sur la durée.

@thbar
Copy link
Contributor Author

thbar commented Sep 12, 2024

J'ai contacté RATP Dev (cc @Brewennn sur ouverture de flux SIRI) pour demander + de précisions techniques.

@thbar thbar self-assigned this Sep 12, 2024
@AntoineAugusti
Copy link
Member

Sans faire de "y'a qu'à faut qu'on", il sera peu fiable d'utiliser la description pour ça sur la durée.

#4070 est le ticket qui aborde ce problème. Je ne crois pas que nous @etalab/transport-tech avait identifié une résolution correcte et planifié le travail requis, qui s'annonce pas facile

@thbar
Copy link
Contributor Author

thbar commented Sep 24, 2024

Discuté avec @Brewennn ce jour ; la ressource est bien marquée "SIRI" au niveau du format. Je vais réfléchir à un patch, en prenant en compte #4070 (merci @AntoineAugusti).

@thbar
Copy link
Contributor Author

thbar commented Sep 25, 2024

On change notre fusil d'épaule, on a contacté RATP Dev pour modifier la description dans l'immédiat. Je garde ouvert pour recetter à la fin.

@thbar
Copy link
Contributor Author

thbar commented Sep 25, 2024

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

No branches or pull requests

2 participants