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

[TECH] Suppression des paramètres non utilisés de l'algo flash (PIX-15475). #10678

Conversation

alexandrecoin
Copy link
Contributor

@alexandrecoin alexandrecoin commented Nov 28, 2024

🍂 Problème

Du nettoyage a été fait dans les paramètres passés à l’algorithme de déroulé dans le cadre de la simulation. PIX-15353 - PIX-15213 - PIX-15316
Ce nettoyage n’a toutefois pas concerné ces mêmes paramètres de l’algo flash afin de pouvoir transmettre la nouvelle version du simulateur le plus rapidement possible à l'équipe Data.

Maintenant que le simulateur est prêt, nous pouvons commencer à supprimer certains de ces mêmes paramètres qui se retrouvent non utilisés (valeurs nulle en BDD) à la fois dans le code, ainsi que les colonnes en BDD associées.

🌰 Proposition

Nous supprimons les occurrences et les colonnes en BDD des paramètres :

  • doubleMeasuresUntil
  • warmUpLength
  • forcedCompetences
  • minimumSuccessRateRanges
  • variationPercentUntil

🎃 Remarques

🪵 Pour tester

  • Vérifier en BDD que les 5 colonnes mentionnées ci-dessus ont disparu de la table flash-algorithm-configurations
  • Se connecter à Pix-Admin en tant que [email protected]
  • Créer une nouvelle configuration d'algorithme dans l'onglet dédié de Pix-Admin
  • Créer une session de certification V3 et y ajouter des candidats afin de pouvoir tester différents procédés de tests (avec et sans dégradation, levée d'alerte)
  • Passer le(s) test(s) de certification
  • Vérifier que tout fonctionne toujours comme il faut (dégradation, scoring, finalisation, publication)

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@alexandrecoin alexandrecoin force-pushed the pix-15475-remove-unused-parameters-in-flash-algorithm branch from 03ca3c0 to 818ae1f Compare November 28, 2024 15:38
@alexandrecoin alexandrecoin marked this pull request as ready for review November 28, 2024 15:46
@alexandrecoin alexandrecoin requested a review from a team as a code owner November 28, 2024 15:46
@alexandrecoin alexandrecoin force-pushed the pix-15475-remove-unused-parameters-in-flash-algorithm branch 2 times, most recently from d587136 to df32f27 Compare November 29, 2024 07:42
@yaf
Copy link
Member

yaf commented Dec 2, 2024

Reste la dégradation à vérifier. Nous avons testé avec trop peu de question. Il faut sans doute se mettre au-dessus de 20 ou un chiffre configuré quelque part.

@alexandrecoin alexandrecoin force-pushed the pix-15475-remove-unused-parameters-in-flash-algorithm branch from df32f27 to e5dd96f Compare December 3, 2024 08:05
Copy link
Contributor

@Steph0 Steph0 left a comment

Choose a reason for hiding this comment

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

La section a tester n'est pas assez detaillee pour me permettre d'etre serein dans ma revue, je manque de precisions pour etre certain que je regarde bien ce que l'auteur de la PR veut me faire verifier.

Mon approve depend donc de la validation suivante sur la degradation

je me rends compte que je sais pas comment verifier que la degradation est bien passee sur un candidat. Je vois que ca prend les answers pour simuler le RAF et retourner juste la nouvelle capacite.

Comment je peux constater un impact de la degradation pour un candidat donnee ?

Est-ce que la demarche suivante est bonne :

  • Je prends le dernier certification-challenge-capacities du dernier certification-challenges
    • sachant que les "je passe" ne produisent apparemment pas de certification-challenge-capacities
  • Je regarde sa capacity
  • Je constate que cette derniere valeur ne correspond pas au score pix de mon assessment-result apres scoring
    • c'est a dire que mon score Pix est de 848 au final, alors que le simulateur sur Pix Admin me dit que pour
      • Capacité : 7.9262137
      • Score : 893

@alexandrecoin alexandrecoin force-pushed the pix-15475-remove-unused-parameters-in-flash-algorithm branch from e5dd96f to e82c8f9 Compare December 4, 2024 13:26
@alexandrecoin alexandrecoin force-pushed the pix-15475-remove-unused-parameters-in-flash-algorithm branch from e82c8f9 to 4143c8f Compare December 4, 2024 13:33
@alexandrecoin alexandrecoin added 🚀 Ready to Merge Func Review OK PO validated functionally the PR labels Dec 4, 2024
@pix-service-auto-merge pix-service-auto-merge merged commit 18c040e into dev Dec 4, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants