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

Jump to bottom

[FEATURE] Flash : Calculer le nombre de Pix obtenus par inférence (PIX-6719) #5458

Merged
merged 2 commits into from Jan 6, 2023
Merged

[FEATURE] Flash : Calculer le nombre de Pix obtenus par inférence (PIX-6719) #5458

merged 2 commits into from Jan 6, 2023

Conversation

nlepage
Copy link
Member

@nlepage nlepage commented Jan 5, 2023 •
edited by Libouk

🎄 Problème

Pour le moment le calcul du score Pix flash n'inclut que les Pix obtenus directement, mais pas les Pix obtenus par inférence.

🎁 Proposition

  • Ré-utiliser la méthode getNonAnsweredChallenges présente dans le fichier flash.js. La renommer en getChallengesForNonAnsweredSkills

  • Créer une fonction qui doit calculer le score Pix obtenu par inférence.

    • Elle prend en entrée des challenges compatibles avec le calcul flash et la capacité de l’utilisateur.
    • Elle filtre les challenges pour garder uniquement ceux dont la capacité minimum est inférieure ou égale à la capacité de l’utilisateur.
    • Elle renvoie la somme des scores Pix associés aux acquis de ces challenges.

  • Intégrer le résultat de cette fonction du total du score Pix obtenu par inférence au total des scores Pix obtenus en direct.

🌟 Remarques

N/A

🎅 Pour tester

Pas de test utilisateur possible pour le moment.

@nlepage nlepage added 👀 Tech Review Needed cross-team Toutes les équipes de dev labels Jan 5, 2023
@pix-service
Copy link
Contributor

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

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

@nlepage nlepage force-pushed the pix-6719-calculate-pix-score-by-inference branch 2 times, most recently from afca0d3 to 0fb5db6 Compare January 5, 2023 14:58
@nlepage nlepage force-pushed the pix-6719-calculate-pix-score-by-inference branch from 0fb5db6 to 6f2eb16 Compare January 5, 2023 15:02
@nlepage nlepage force-pushed the pix-6719-calculate-pix-score-by-inference branch 2 times, most recently from 3797009 to bae2929 Compare January 6, 2023 09:32
@nlepage nlepage force-pushed the pix-6719-calculate-pix-score-by-inference branch 2 times, most recently from 0cbb7f2 to 4a9cd7d Compare January 6, 2023 10:50
nlepage and others added 2 commits January 6, 2023 13:23
@nlepage @mcampourcy @Libouk
♻️ api: Rename getNonAnsweredChallenges to getChallengesForNonAnswere…
caedee1 
…dSkills

Co-authored-by: Mireille Campourcy <mireille.campourcy@pix.fr>
Co-authored-by: Quentin Lebouc <quentin.lebouc@pix.fr>
@nlepage @mcampourcy @Libouk
✨ api: Calculate inferred Pix score
e3b5bbe 
Co-authored-by: Mireille Campourcy <mireille.campourcy@pix.fr>
Co-authored-by: Quentin Lebouc <quentin.lebouc@pix.fr>
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-6719-calculate-pix-score-by-inference branch from 4a9cd7d to e3b5bbe Compare January 6, 2023 13:23
@pix-service-auto-merge pix-service-auto-merge merged commit 80bff29 into dev Jan 6, 2023
@pix-service-auto-merge pix-service-auto-merge deleted the pix-6719-calculate-pix-score-by-inference branch January 6, 2023 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mcampourcy mcampourcy mcampourcy approved these changes

@Libouk Libouk Libouk approved these changes

Assignees

@mcampourcy mcampourcy

@nlepage nlepage

@Libouk Libouk

Labels
cross-team Toutes les équipes de dev 🚀 Ready to Merge Tech Review OK
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@nlepage @pix-service @mcampourcy @Libouk @pix-service-auto-merge