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

Move new suggestor to production #1623

Closed
ClementChapalain opened this issue Nov 4, 2019 · 15 comments
Closed

Move new suggestor to production #1623

ClementChapalain opened this issue Nov 4, 2019 · 15 comments

Comments

@ClementChapalain
Copy link
Contributor

cf. #1549

@rmelisson
Copy link
Contributor

rmelisson commented Nov 5, 2019

Nous sommes en train de faire des changements sur le suggesteur, en vrac :

  • Gestion de l’accentuation
  • Fuzzy matching (fautes de frappe)
  • Utilisation d’une pondération pour ordonner les résultats
  • Cleaning des suggestions (déduplication, verification orthographique…)
  • Infix suggestions (quand une suggestion match “au milieu”: ‘demi’ -> préavis demission)

Quelques limitations connues :

@rmelisson
Copy link
Contributor

@ClementChapalain
Copy link
Contributor Author

c'est top ! quelques suggestions qui ne paraissent pas pertinentes / avec des fautes :

  • coiffeuse 76000
  • 1 mois apres l'entretient disciplinaire-licenciment

@mhavel
Copy link

mhavel commented Nov 5, 2019

image

"belgique", "une personne" : pas top.

Mais sinon c'est rapide, très souvent pertinent et varié. Excellent

@rmelisson
Copy link
Contributor

Je pense que domicile Belgique fait sens pour les personnes résidant en Belgique et travaillant en France.
Après ça concerne la liste initiale, on va corriger ça indépendamment : #1631

@virginielastisneres
Copy link
Contributor

Je n'ai pas le même résultat que je tape :

  • licencement
  • licencment
  • ou licenciement

@virginielastisneres
Copy link
Contributor

image
suggestion pas très pertinente

@virginielastisneres
Copy link
Contributor

virginielastisneres commented Nov 5, 2019

bizarre ahah
image

J'en profite pour vous demander s'il est possible de faire correspondre syntec à la convention collective des bureaux techniques ? :)

@lionelB
Copy link
Contributor

lionelB commented Nov 5, 2019

J'en profite pour vous demander s'il est possible de faire correspondre syntec à la convention collective des bureaux techniques ? :)

a voir si on introduit la gestion de synonyme

@virginielastisneres
Copy link
Contributor

mise a pide ne donne pas la même chose que mise à pied

@virginielastisneres
Copy link
Contributor

suggestion de mots avec des fautes d'orthographe
j

@rmelisson
Copy link
Contributor

rmelisson commented Nov 5, 2019

@virginielastisneres merci pour les retours :

  • pour le cas des suggestions différentes entre les requêtes avec ou sans faute, c'est normal car l'entrée est différente, licencment == licenciement pour toi, mais le suggester cherche tous les parcours "à distance de faute", par exemple licence, donc c'est normal de ne pas avoir exactement la même réponse
  • aussi, il faut prendre en compte le 'parcours' de requête, car les suggestions sont présentées au fur et à mesure (dans l'exemple licenciement, les suggestions sont cohérentes pour les 10 premiers caractères)
  • en qui concerne les suggestions erronées (fautes, truc bizarres...), ça sera traité dans Consolidation des suggestions via analyse des logs  #1631 mais on va déjà enlever ce qui a été remonté ici
  • enfin pour ce qui est des synonymes, et plus généralement l'interpretation des requêtes (genre mapping query => question en language naturel ou query => synonymes...), je crée un nouveau ticket Interprétation des requêtes #1641 pour garder ça en tête, mais c'est pas dans le scope de cette update

@virginielastisneres
Copy link
Contributor

super merci @rmelisson :)

@lionelB
Copy link
Contributor

lionelB commented Nov 15, 2019

@ClementChapalain on peut fermer puisque la PR #1549 à été mergé ?

@ClementChapalain
Copy link
Contributor Author

oui !

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

5 participants