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

Suggester - stopwords inactifs #1161

Closed
3 tasks done
AnneHuSKa opened this issue Oct 23, 2024 · 11 comments
Closed
3 tasks done

Suggester - stopwords inactifs #1161

AnneHuSKa opened this issue Oct 23, 2024 · 11 comments
Assignees
Labels
Type: Bug Something isn't working

Comments

@AnneHuSKa
Copy link

AnneHuSKa commented Oct 23, 2024

Describe the bug

En testant le suggester des PCS dont les stopwords contiennent DANS, j'ai parmi les échos :
image

Or dans est un stopwords donc l'écho ne devrait pas sortir me semble-t-il ?

But ultérieur : ajouter des stoopwords sur les communes et les département également mais à court terme, permettre une recherche PCS plus efficace.

To Reproduce

Question SUGGPCS de lvpb6ks2 en prod.

Expected behavior

Avoir moins d'échos et plus pertinents

Screenshots

Version where the bug appeared

Lunatic : 3.4.4 ou avant

Browser where the bug was tested

  • Firefox

Additional context

Tâches

@AnneHuSKa AnneHuSKa added the Type: Bug Something isn't working label Oct 23, 2024
@JulienCarmona JulienCarmona removed their assignment Oct 25, 2024
@JulienCarmona
Copy link
Contributor

Hello @laurentC35 ,

Besoin d'une estimation pour le prochain sprint, car cela risque d'être assez urgent

@laurentC35
Copy link
Contributor

laurentC35 commented Oct 31, 2024

On a oublié plein de méthode dans le refacto vers la 3.0
cf: https://github.com/InseeFr/Lunatic/blob/2.6/src/utils/suggester-workers/commons-tokenizer/create-entity-tokenizer.js#L19

voir le poc pour voir ce qu'on peut utiliser : #828

Actuellement dans la 3.0: https://github.com/InseeFr/Lunatic/tree/3.0/src/utils/search

Recherche à partir de seulement 3 caractères (>2): https://github.com/InseeFr/Lunatic/blob/3.0/src/utils/search/SearchMinisearch.ts#L44

@laurentC35
Copy link
Contributor

laurentC35 commented Oct 31, 2024

@JulienCarmona Au vu de l'estimation, on pourrait faire appel à Jonathan.
En revanche, en tant qu'équipe on souhaiterait avoir la maîtrise de cette partie là (seule Jonathan et un bout de Renaud là).
L'estimation prend en compte la montée en compétence de l'équipe sur le fonctionnement des suggesters dans Lunatic.

On pourrait donc réduire le coût en travaillant de pair avec lui tout en montant en compétence sur le sujet suggester.

@JulienCarmona
Copy link
Contributor

@JulienCarmona Au vu de l'estimation, on pourrait faire appel à Jonathan. En revanche, en tant qu'équipe on souhaiterait avoir la maîtrise de cette partie là (seule Jonathan et un bout de Renaud là). L'estimation prend en compte la montée en compétence de l'équipe sur le fonctionnement des suggesters dans Lunatic.

On pourrait donc réduire le coût en travaillant de pair avec lui tout en montant en compétence sur le sujet suggester.

Très bonne idée !

  • Est-ce que tu es autonome pour prendre contacte avec Jonathan et lui demander de faire le boulot avec vous ? Ou faut qu'on s'organise avec Anne ?
  • Est ce que les 21pts ca comprends le taff de Jonathan ? Si jamais il nous aide on pourrait déduire sa charge de l'estimation car il est pas dans notre sprint en tant qu'acteur. A discuter avec Antoine.

@JulienCarmona
Copy link
Contributor

A redécouper

@QRuhier
Copy link
Contributor

QRuhier commented Nov 25, 2024

@QRuhier
Copy link
Contributor

QRuhier commented Nov 26, 2024

à reprendre : les stopwords sont actuellement complètement exclus de la recherche (une recherche "dans" ne trouvera aucun echo)

@lailabent
Copy link

Done suite à recette dans le storybook par rapport à la demande initiale
Côté métier on va étudier la note récap faite par @QRuhier , on ne s'interdit pas de faire de nouvelles demandes ultérieurement

@lailabent lailabent reopened this Nov 27, 2024
@QRuhier QRuhier closed this as completed Nov 27, 2024
@JulienCarmona
Copy link
Contributor

Release 3.4.8 a mettre dans les orchestrateurs

@JulienCarmona
Copy link
Contributor

Remarque :

  • stopWords ne semble pas marcher sur les mots en deuxième position. Côté métier on est ok avec ce comportement donc si personne se plaint on laisse comme ça

@QRuhier
Copy link
Contributor

QRuhier commented Dec 2, 2024

C'est à dire ? Preneur si tu as un exemple qu'on puisse se le noter ;-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants