Skip to content

[FEATURE] Ajout du score global dans les resultats Parcoursup (PIX-16548). #11448

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

Merged
merged 6 commits into from
Feb 19, 2025

Conversation

Steph0
Copy link
Contributor

@Steph0 Steph0 commented Feb 18, 2025

🥞 Problème

On veut ajouter le niveau global de Pix dans les remontees Parcoursup.

Il est admis que pas de traductions pour le niveau zero
Prendre en compte que le score depend de la configuration des mailles et que c'est la maille qui donne le niveau

🥓 Proposition

  • Creer un usecase qui fait appel au service qui permet de retrouver la maille a partir du score Pix
  • Ajouter la traduction equivalente de cette maille en un niveau global, dans la reponse Parcoursup

🧃 Remarques

  • Il va falloir voir avec l'equipe DATA car on utilise pas la base froide
  • Le service qui est dans 'scoring' (contexte qui doit disparaitre) n'a pas ete migre vers result dans cette PR, mais il faudrait le faire un jour
    • @alexandrecoin est en phase de deplacer dans results le simulateur associe

😋 Pour tester

  • Dans le DATAMART, il faut reperer les cas suivants, et verifier le champ globalLevel
    • certification valide
      • doit donner lieu a une traduction en Niveau X selon la maille auquel appartient le score
    • certification rejete avec un score < niveau 1
      • ne doit pas donner de global level (pas de niveau zero)
    • certification annulee avec un pix score > maille zero
      • ne doit pas donner de globalLevel meme si il y a une maille autre que zero

@Steph0 Steph0 self-assigned this Feb 18, 2025
@Steph0 Steph0 requested a review from a team as a code owner February 18, 2025 17:14
@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 :

@Steph0 Steph0 force-pushed the PIX-16548-find-mesh-level-for-a-pix-score branch from f1d870d to f922d4c Compare February 18, 2025 18:23
@Steph0 Steph0 added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally team-certif labels Feb 18, 2025
@Steph0 Steph0 force-pushed the PIX-16548-find-mesh-level-for-a-pix-score branch from f922d4c to 580a5e0 Compare February 19, 2025 08:57
@yaf yaf added the Parcoursup label Feb 19, 2025
@Steph0 Steph0 force-pushed the PIX-16548-find-mesh-level-for-a-pix-score branch from 580a5e0 to f3ac628 Compare February 19, 2025 13:03
pixScore: certificationResult.pixScore,
certificationDate: certificationResult.certificationDate,
competences: certificationResult.competences,
globalLevel: globalMeshLevel.getLevelLabel(translate),
Copy link
Member

@AndreiaPena AndreiaPena Feb 19, 2025

Choose a reason for hiding this comment

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

Possible de le mettre après le pixScore?
Sinon là il est affiché tout en bas après le tableau des compétences.

Il est également affiché en anglais (mon navigateur est en anglais), pour s'éviter ça peut-on forcer en français pour Parcoursup et laisser la liberté de la langue pour un autre usage ?
(vu ensemble, on ne force rien pour Parcoursup mais on précisera cet élément dans la documentation)

Capture d’écran 2025-02-19 à 14 34 59

Copy link
Member

@AndreiaPena AndreiaPena left a comment

Choose a reason for hiding this comment

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

select "national_student_id","pix_score" from "xxx" where "pix_score"<{pixScore} group by "national_student_id", "pix_score" order by "pix_score" desc;

Cas du PixScore à moins de 64 pix : je m'attends à avoir un globalLevel vide

=> J'obtiens ce retour. Il ne trouve pas la clé donc il renvoit ceci.
Capture d’écran 2025-02-19 à 14 42 00

Cas du PixScore entre 64 et 127 pix : je m'attends à avoir un globalLevel "Novice 1"

=> Ok ✅
Capture d’écran 2025-02-19 à 14 52 47
Capture d’écran 2025-02-19 à 14 53 28

Cas du PixScore entre 128 et 255 pix : je m'attends à avoir un globalLevel "Novice 2"

=> Ok ✅
Capture d’écran 2025-02-19 à 14 51 04
Capture d’écran 2025-02-19 à 14 51 17

Cas du PixScore entre 256 et 383 pix : je m'attends à avoir un globalLevel "Indépendant 1"

=> Ok ✅
Capture d’écran 2025-02-19 à 15 00 53
Capture d’écran 2025-02-19 à 15 01 01
Capture d’écran 2025-02-19 à 15 02 14
Capture d’écran 2025-02-19 à 15 01 01

Cas du PixScore entre 640 et 767 pix : je m'attends à avoir un globalLevel "Avancé 2"

=> Ok ✅
Capture d’écran 2025-02-19 à 15 07 20
Capture d’écran 2025-02-19 à 15 07 26

Cas du PixScore entre 768 et 895 pix : je m'attends à avoir un globalLevel "Expert 1"

=> Ok ✅
Capture d’écran 2025-02-19 à 15 03 55
Capture d’écran 2025-02-19 à 15 04 03

@Jeyffrey Jeyffrey force-pushed the PIX-16548-find-mesh-level-for-a-pix-score branch from 4d24884 to fcf30db Compare February 19, 2025 15:16
const globalMeshLevel = await usecases.getGlobalMeshLevel({
pixScore: certificationResult.pixScore,
certificationDate: certificationResult.certificationDate,
i18n: request.i18n,
Copy link
Member

@AndreiaPena AndreiaPena Feb 19, 2025

Choose a reason for hiding this comment

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

getI18n(FRENCH_FRANCE).__

@Steph0
Copy link
Contributor Author

Steph0 commented Feb 19, 2025

Test func pour niveau 0 OK ✅

image

Steph0 and others added 6 commits February 19, 2025 16:43
Co-authored-by: Andreia Pena <andreia.pena@pix.fr>
Co-authored-by: Geoffroy Begouaussel <geoffroy.begouaussel@pix.fr>
Co-authored-by: Geoffroy Begouaussel <geoffroy.begouaussel@pix.fr>
* 32 car. required by lib hapi/yar

Co-authored-by: Andreia Pena <andreia.pena@pix.fr>
Co-authored-by: Benjamin Petetot <516360+bpetetot@users.noreply.github.com>
Co-authored-by: Geoffroy Begouaussel <geoffroy.begouaussel@pix.fr>
Co-authored-by: Yannick François <yannick.francois@pix.fr>
Co-authored-by: Andreia Pena <andreia.pena@pix.fr>
Co-authored-by: Geoffroy Begouaussel <geoffroy.begouaussel@pix.fr>
@Steph0 Steph0 force-pushed the PIX-16548-find-mesh-level-for-a-pix-score branch from fcf30db to 9d4af6f Compare February 19, 2025 15:43
@Steph0 Steph0 added the Func Review OK PO validated functionally the PR label Feb 19, 2025
@pix-service-auto-merge pix-service-auto-merge merged commit bc018d0 into dev Feb 19, 2025
9 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the PIX-16548-find-mesh-level-for-a-pix-score branch February 19, 2025 16:01
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