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

[BUGFIX] Récupérer uniquement les compétences sur le référentiel Pix Coeur pour le scoring V3 (PIX-16272). #11268

Merged

Conversation

alexandrecoin
Copy link
Contributor

@alexandrecoin alexandrecoin commented Jan 29, 2025

🥞 Problème

A l’heure actuelle, dans l'étape de scoring, nous récupérons l’entièreté du référentiel de Pix (Coeur + 1D + Complémentaires, etc) et donc des compétences qui le composent.

Lors du calcul de niveau par compétence, nous passons par la méthode V3CertificationScoring.fromConfigurations()

Cette méthode se base sur la configuration de score par compétence enregistrée en BDD et dont le schéma est comme suit :

[
  { competence: "1.1", values: [{competenceLevel: 0, bounds: {min: -8, max: -6}, {competenceLevel: 1, bounds: {min: -6, max: -4}, ...},
  { competence: "1.2", values: [{competenceLevel: 0, bounds: {min: -8, max: -5.5}, {competenceLevel: 1, bounds: {min: -5.5, max: -3}, ...}, 
  ...
]

A partir de cette liste, nous tentons de calculer le niveau par compétence du candidat.
Pour cela, nous retrouvons la compétence en question en se basant sur le code de chaque compétence (ex: competence: "1.1") utilisé dans la configuration ci-dessus, de façon à ce qu’il corresponde à l’un des codes de compétences passés en paramètre de la fonction sous le nom de competenceList

Or, ce code peut être retrouvé dans plusieurs référentiels de Pix et ne peut donc être utilisé pour trouver la compétence en question.
Ceci entraine l’enregistrement de competence-marks dont l’ origin (= réferentiel) est erronée.

🥓 Proposition

Récupérer uniquement les compétences provenant du référentiel Pix Coeur

🧃 Remarques

😋 Pour tester

  • Créer une session V3
  • Ajouter un candidat
  • Passer la certification
  • Une fois le test complété, vérifier en BDD que l'origin des competences-marks qui ont été créées soit uniquement Pix

@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 :

Copy link
Member

@yaf yaf left a comment

Choose a reason for hiding this comment

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

👌 trop bien. :shipit:

@yaf yaf added Func Review OK PO validated functionally the PR 👀 Func Review Needed Need PO validation for this functionally and removed 👀 Func Review Needed Need PO validation for this functionally Func Review OK PO validated functionally the PR labels Jan 30, 2025
@yaf
Copy link
Member

yaf commented Jan 30, 2025

✔️ func review dev OK !

Requête SQL pour vérifier en base

with tutu as ( select "competences".origin
from "competence-marks" 
inner join "learningcontent"."competences" on "competence-marks"."competenceId" = "learningcontent"."competences".id
inner join "assessment-results" as ares on ares.id = "competence-marks"."assessmentResultId"
inner join "assessments" as ass on ass.id = ares."assessmentId"
inner join "certification-courses" as cc on cc.id = ass."certificationCourseId"
inner join "sessions" on "sessions".id = cc."sessionId"
where "sessions".version = 3
and ass.type = 'CERTIFICATION' )
select count(1), tutu.origin
from tutu 
group by tutu.origin;

@Agnes-V
Copy link
Contributor

Agnes-V commented Jan 30, 2025

Hello @1024pix/team-certification ! 👋🏼
J'ai testé en RA avec la session n° 7412 avec 2 candidats, l'un passant une certification Pix seule (certification n°128635) et l'autre passant une double certification Pix/CléA Num (certification n°128636).
Est-ce que vous pouvez aller vérifier en BDD que l'origin des competences-marks qui ont été créées soit uniquement Pix svp ? 🙏🏼
Dispo pour passer sur un meet/slack/cequevousvoulez 👀

@Agnes-V Agnes-V added Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed Need PO validation for this functionally labels Jan 30, 2025
@pix-service-auto-merge pix-service-auto-merge force-pushed the PIX-16272-fix-scoring-on-competences-marks-pairing branch from 20da515 to 1e3182f Compare January 30, 2025 14:23
@pix-service-auto-merge pix-service-auto-merge merged commit b6b08c4 into dev Jan 30, 2025
8 of 10 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the PIX-16272-fix-scoring-on-competences-marks-pairing branch January 30, 2025 14:30
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.

7 participants