Services/loterre annotate/loterre annotate first by stephane54 · Pull Request #444 · Inist-CNRS/web-services

stephane54 · 2026-06-15T07:48:34Z

No description provided.

parmentf

Une petite suggestion (pour une typo) et des remarques diverses...

parmentf · 2026-06-15T09:28:52Z

+post.requestBody.content.application/json.schema.$ref = #/components/schemas/JSONStream
+post.requestBody.required = true
+post.responses.default.content.application/json.schema.$ref = #/components/schemas/JSONStream
+post.summary = Produit la liste des termes Loterre identifiés dans le texte ou rend le texte annoté


❓ question: Il y a un paramètre pour sélectionner le comportement de la route (produire la liste des termes identifiés / rendre le texte annoté) ?

Ou bien c'est juste une manière différente d'exprimer la même chose ?
L'exemple indique plutôt le premier comportement.‏

parmentf · 2026-06-15T09:30:14Z

+post.requestBody.content.application/json.schema.$ref = #/components/schemas/JSONStream
+post.requestBody.required = true
+post.responses.default.content.application/json.schema.$ref = #/components/schemas/JSONStream
+post.summary = Produit la liste des termes Loterre identifiés dans le texte ou rend le texte annoté


❓ question: ‏Même question que pour la version en...

parmentf · 2026-06-15T09:43:17Z

+            }
+        }
+    ],
+    "paths": {


🪳 : nitpick (non-blocking): ‏D'habitude on ne crée pas nous-mêmes les champs paths et components du swagger, c'est le serveur ezs qui le fait.
Il est possible que ce fichier surcharge ce que produirait le serveur ezs.
L'important c'est que si tu gardes cette manière de faire, il faut maintenir cette partie aussi (au lieu de la générer via les métadonnées des fichiers .ini).

Mais tout ça m'a l'air très propre.

parmentf · 2026-06-15T09:47:17Z

+ENV LOTERRE_REGISTRY=/app/public/configs/registry.yaml
+
+RUN apt-get update && \
+    apt-get -y --no-install-recommends install git && \


🪳 : nitpick (non-blocking): ‏La bonne pratique pour l'installation d'un paquet apt dans un Dockerfile est de mettre sa version complète (même si je comprends que pendant le développement, c'est plus pratique de ne pas la préciser).

C'est principalement pour une question de reproductibilité (et malheureusement, ça force à une maintenance plus pénible pour les versions suivantes, mais c'est déjà ce qui est fait en ligne 7 pour la première étape).

Co-authored-by: François Parmentier <francois.parmentier@gmail.com>

stephane54

Corrigé

parmentf

OK. Tu peux maintenant faire une nouvelle version:

npm -w services/loterre-annotate version major

Ça provoquera la construction de l'image et ça la poussera sur Docker Hub.
Il n'y aura plus qu'à renseigner la machine et le port dans swagger.json avant que je puisse merger la branche, et créer l'URL qui pointera vers le bon container.

Ah, mais je vois que ton swagger.json n'a plus les lignes à modifier:

        {
            "url": "http://vptermsuite.intra.inist.fr:49157/",
            "description": "Latest version for production",
            "x-profil": "Standard"
        }

Il faut y mettre le bon port (et ne pas oublier le champ x-profil, sinon notre script de publication ne fonctionnera pas).

parmentf

Ce qui est étonnant, c'est que des tests soient quand même passés.

Co-authored-by: François Parmentier <francois.parmentier@gmail.com>

parmentf · 2026-06-19T07:30:28Z

Bon, ben maintenant il faut faire une version patch.

parmentf

Je ne vois pas ce qui coince dans la construction de l'image (a priori c'est la récupération du dictionary, mais pourquoi alors que la version précédente était passée.).

parmentf · 2026-06-24T11:57:09Z

+
+#### URL du service
+
+> https://loterre-annotate.2.0.2/v1/CODE_LANGUE/loterre-annotate/2.0.2?loterreID=CODE_VOC


🪳 : nitpick (non-blocking): ‏Bad URL

Suggested change

> https://loterre-annotate.2.0.2/v1/CODE_LANGUE/loterre-annotate/2.0.2?loterreID=CODE_VOC

> https://loterre-annotate.services.istex.fr/v1/CODE_LANGUE/loterre-annotate/annotate?loterreID=CODE_VOC

parmentf · 2026-06-24T11:58:02Z

+### Requête
+
+```bash
+curl -X POST 'https://loterre-annotate.2.0.2/v1/en/loterre-annotate/2.0.2?loterreID=P66' \


🪳 : nitpick (non-blocking): ‏Bad URL

Suggested change

curl -X POST 'https://loterre-annotate.2.0.2/v1/en/loterre-annotate/2.0.2?loterreID=P66' \

curl -X POST 'https://loterre-annotate.services.istex.fr/v1/en/loterre-annotate?loterreID=P66' \

parmentf · 2026-06-24T12:48:49Z

+        },
+        {
+            "url": "https://loterre-annotate.services.istex.fr",
+            "description": "Service de production"


☑️ todo: ‏Add x-profil standard

Suggested change

"description": "Service de production"

"description": "Service de production",

"x-profil": "Standard"

Did you use an AI Assistant, it keeps removing that line...

stephane54 added 8 commits June 12, 2026 13:53

n

e3b5a99

n

09f6df5

n

8635501

n

14019d4

n

0fc6603

1 push

5534db1

1 fst push

6e11d70

push one

cd64601

parmentf requested changes Jun 15, 2026

View reviewed changes

stephane54 and others added 5 commits June 16, 2026 09:29

Update annotation description in annotate.ini

edc5cb8

Update summary in annotate.ini for clarity

67a4437

Update annotate.ini

8a447cd

Update Dockerfile

d5c14c2

Apply suggestion from @parmentf

5eb965f

Co-authored-by: François Parmentier <francois.parmentier@gmail.com>

stephane54 commented Jun 16, 2026

View reviewed changes

parmentf reviewed Jun 16, 2026

View reviewed changes

parmentf assigned stephane54 Jun 16, 2026

stephane54 added 2 commits June 16, 2026 16:36

good

b4c5516

release ws-loterre-annotate@2.0.0

f301780

parmentf requested changes Jun 17, 2026

View reviewed changes

Comment thread services/loterre-annotate/Dockerfile Outdated

Update services/loterre-annotate/Dockerfile

1cb1186

Co-authored-by: François Parmentier <francois.parmentier@gmail.com>

stephane54 added 7 commits June 22, 2026 08:31

release ws-loterre-annotate@2.0.1

b31c4d1

release loterre-annotate@2.0.1

faddffa

release loterre-annotate@2.0.2

869522a

release ws-loterre-annotate@2.0.3

557dba1

release ws-loterre-annotate@2.0.4

e86ab09

release ws-loterre-annotate@2.0.5

95aac37

release ws-loterre-annotate@2.0.6

50ea29e

parmentf requested changes Jun 24, 2026

View reviewed changes


		#### URL du service

		> https://loterre-annotate.2.0.2/v1/CODE_LANGUE/loterre-annotate/2.0.2?loterreID=CODE_VOC

	> https://loterre-annotate.2.0.2/v1/CODE_LANGUE/loterre-annotate/2.0.2?loterreID=CODE_VOC
	> https://loterre-annotate.services.istex.fr/v1/CODE_LANGUE/loterre-annotate/annotate?loterreID=CODE_VOC

	curl -X POST 'https://loterre-annotate.2.0.2/v1/en/loterre-annotate/2.0.2?loterreID=P66' \
	curl -X POST 'https://loterre-annotate.services.istex.fr/v1/en/loterre-annotate?loterreID=P66' \

	"description": "Service de production"
	"description": "Service de production",
	"x-profil": "Standard"

Uh oh!

Conversation

stephane54 commented Jun 15, 2026

Uh oh!

parmentf left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

parmentf Jun 15, 2026

Choose a reason for hiding this comment

Uh oh!

parmentf Jun 15, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

parmentf Jun 15, 2026

Choose a reason for hiding this comment

Uh oh!

parmentf Jun 15, 2026

Choose a reason for hiding this comment

Uh oh!

parmentf Jun 15, 2026

Choose a reason for hiding this comment

Uh oh!

stephane54 left a comment

Choose a reason for hiding this comment

Uh oh!

parmentf left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

parmentf left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

parmentf commented Jun 19, 2026

Uh oh!

parmentf left a comment

Choose a reason for hiding this comment

Uh oh!

parmentf Jun 24, 2026

Choose a reason for hiding this comment

Uh oh!

parmentf Jun 24, 2026

Choose a reason for hiding this comment

Uh oh!

parmentf Jun 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

parmentf left a comment •

edited

Loading

parmentf Jun 24, 2026 •

edited

Loading