Skip to content

[text-lucene] create service#407

Merged
parmentf merged 9 commits into
mainfrom
services/text-lucene/create-service
Mar 18, 2026
Merged

[text-lucene] create service#407
parmentf merged 9 commits into
mainfrom
services/text-lucene/create-service

Conversation

@leogail
Copy link
Copy Markdown
Collaborator

@leogail leogail commented Mar 13, 2026

No description provided.

Copy link
Copy Markdown
Contributor

@parmentf parmentf left a comment

Choose a reason for hiding this comment

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

C'est un bon début.
Faudra ajouter les tests.
Évidemment, il faut modifier le package.json pour que ça marche en local, normalement.
Et le .env (avec la clé).

Comment thread services/text-lucene/config.json Outdated
Comment thread services/text-lucene/package.json Outdated
Comment thread services/text-lucene/README.md Outdated
Comment thread services/text-lucene/swagger.json Outdated
Comment thread README.md
- [data-computer](./services/data-computer) [![Docker Pulls](https://img.shields.io/docker/pulls/cnrsinist/ws-data-computer.svg)](https://hub.docker.com/r/cnrsinist/ws-data-computer/)
- [data-graph](./services/data-graph) [![Docker Pulls](https://img.shields.io/docker/pulls/cnrsinist/ws-data-graph.svg)](https://hub.docker.com/r/cnrsinist/ws-data-graph/)
- [data-homogenise](./services/data-homogenise) [![Docker Pulls](https://img.shields.io/docker/pulls/cnrsinist/ws-data-homogenise.svg)](https://hub.docker.com/r/cnrsinist/ws-data-homogenise/)
- [data-kwsimilarity](./services/data-kwsimilarity) [![Docker Pulls](https://img.shields.io/docker/pulls/cnrsinist/ws-data-kwsimilarity.svg)](https://hub.docker.com/r/cnrsinist/ws-data-kwsimilarity/)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🪳 : nitpick (non-blocking): ‏Bizarre.

D'où ça vient ?!?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Aucune idée, j'ai pourtant bien exécuté un pull avant de créer la branche..

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

J'ai eu la même chose en créant une autre branche #409 (depuis la branche main à jour).
Étrange.

@leogail
Copy link
Copy Markdown
Collaborator Author

leogail commented Mar 13, 2026

Tu as été très rapide pour review François !
En effet j'avais pas modifié les fichiers encore. Mais j'ai un problème vraiment bizarre :
Les requêtes fonctionnent bien en local avec curl mais pas depuis le fichier example.http.

J'opérerai ces modifications lundi..

@leogail leogail added the enhancement New feature or request label Mar 17, 2026
@leogail leogail self-assigned this Mar 17, 2026
Copy link
Copy Markdown
Contributor

@parmentf parmentf left a comment

Choose a reason for hiding this comment

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

Rien d'important à redire.
Pas mal de détails.
Et quelques réflexions.

Comment thread services/text-lucene/v1/istex-search.py Outdated
Comment thread services/text-lucene/v1/istex-search.py
Comment thread services/text-lucene/v1/istex-search.py Outdated
Comment thread services/text-lucene/v1/istex-search.py
Comment thread services/text-lucene/v1/istex-search.py
Comment thread services/text-lucene/v1/istex-search.py Outdated
Comment thread services/text-lucene/v1/istex-search.py Outdated
Comment thread services/text-lucene/swagger.json Outdated
@parmentf parmentf merged commit acc65d8 into main Mar 18, 2026
@parmentf parmentf deleted the services/text-lucene/create-service branch March 18, 2026 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants