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

ETQ usager, l information de durée de traitement des 90 percentile peut être trompeur #9708

Merged

Conversation

mfo
Copy link
Contributor

@mfo mfo commented Nov 14, 2023

contexte

je sais plus ou est le crisps, mais un usager nous remonte qu'on affiche 5 mois de traitement moyen sur une procedure. Or dans le meilleur des cas (30% des dossiers), ca prend 3 mois.

solution

on groupe les données en 3 groupes et appliquons le pencentile sur chacun des groupes :
sur de la donnée de prod on retrouve nos petits

expect(procedure.usual_traitement_time_for_recent_dossiers(30).map {distance_of_time_in_words(_1)}).to eq(["3 mois", "6 mois", "environ un an"])

visuel:

apres : (mon dossier en construction)

Screenshot 2023-12-07 at 9 41 47 AM

apres : (page commencé)

Screenshot 2023-12-07 at 9 29 31 AM

avant :

@mfo mfo marked this pull request as draft November 14, 2023 16:52
@mfo mfo changed the title ETQ usager, l information de durée de traitement des 90 percentile peut être trompeur WIP - ETQ usager, l information de durée de traitement des 90 percentile peut être trompeur Nov 14, 2023
@mfo mfo force-pushed the US/better-estimation-on-treatment-time branch 4 times, most recently from 3d01db6 to 287e64e Compare November 21, 2023 11:34
@mfo mfo changed the title WIP - ETQ usager, l information de durée de traitement des 90 percentile peut être trompeur ETQ usager, l information de durée de traitement des 90 percentile peut être trompeur Nov 21, 2023
@mfo mfo force-pushed the US/better-estimation-on-treatment-time branch from 287e64e to 4d2281d Compare November 21, 2023 13:24
@mfo mfo marked this pull request as ready for review November 21, 2023 14:46
@mfo mfo force-pushed the US/better-estimation-on-treatment-time branch from 4d2281d to c056dbb Compare November 21, 2023 14:47
Copy link
Member

@colinux colinux left a comment

Choose a reason for hiding this comment

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

J'ai fait des propositions d'amélioration de wording pour essayer de réduire l'impression qu'on fait une prédiction/estimation du délai de traitement du dossier. Ce qu'on expose, c'est un constat.

@mfo mfo force-pushed the US/better-estimation-on-treatment-time branch from fe29047 to 385f120 Compare November 28, 2023 13:25
@mfo mfo force-pushed the US/better-estimation-on-treatment-time branch from 76292c6 to 709db08 Compare December 7, 2023 08:19
Martin and others added 2 commits December 7, 2023 09:19
…vise en 3 groupe l'estimation de la duree de traitement
…ction

Co-authored-by: Colin Darie <colin@darie.eu>
@mfo mfo force-pushed the US/better-estimation-on-treatment-time branch 3 times, most recently from 90f3fe3 to abe288a Compare December 7, 2023 08:44
Copy link
Member

@colinux colinux 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 bon pour moi, juste une dernière remarque sans avoir le nez dedans : dans la vue on conditionne l'accordéon avec - if procedure.stats_usual_traitement_time , et dans le component c'est presque la même chose mais pas tout à fait.

Est-ce qu'on a des cas où il pourrait y avoir une différence entre les 2, ou est-ce qu'il yaurait un intérêt de partager la même logique ? Une solution (pas très élégante) serait de faire quelque chose comme :

- estimated_delay_component = Procedure::EstimatedDelayComponent.new(procedure: procedure)
- if estimated_delay_component.render?
  %section.fr-accordion
  ....
  = render estimated_delay_component

@mfo mfo force-pushed the US/better-estimation-on-treatment-time branch from abe288a to 5c00dc1 Compare December 7, 2023 13:33
@mfo mfo enabled auto-merge December 7, 2023 13:34
@mfo mfo force-pushed the US/better-estimation-on-treatment-time branch 4 times, most recently from 7d39b09 to 00cb21c Compare December 7, 2023 14:59
@mfo mfo force-pushed the US/better-estimation-on-treatment-time branch from 00cb21c to a66fd42 Compare December 7, 2023 15:47
@mfo mfo added this pull request to the merge queue Dec 7, 2023
Merged via the queue into demarches-simplifiees:main with commit b28cbd9 Dec 7, 2023
15 checks passed
@mfo mfo deleted the US/better-estimation-on-treatment-time branch December 7, 2023 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants