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

Feat: activation de l'optimisation du modèle avec publiopti #839

Merged
merged 10 commits into from
Feb 27, 2023

Conversation

EmileRolley
Copy link
Contributor

@EmileRolley EmileRolley commented Feb 13, 2023

Related issue: incubateur-ademe/nosgestesclimat#1661

https://deploy-preview-839--nosgestesclimat.netlify.app/simulateur/bilan?PR=1697

Suite à une mise en ligne de l'infrastructure, ici on active effectivement l'optimisation avec plubliopti.

Réduction ~65% du nombre de règles chargées et ~50% du temps de parsing (en preview).

@EmileRolley EmileRolley mentioned this pull request Feb 13, 2023
8 tasks
@netlify
Copy link

netlify bot commented Feb 13, 2023

Deploy Preview for nosgestesclimat ready!

Name Link
🔨 Latest commit 4943454
🔍 Latest deploy log https://app.netlify.com/sites/nosgestesclimat/deploys/63fcea2a9fdb8d000891637b
😎 Deploy Preview https://deploy-preview-839--nosgestesclimat.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@EmileRolley EmileRolley marked this pull request as draft February 13, 2023 16:48
@github-actions
Copy link

github-actions bot commented Feb 14, 2023

Report for the pull request #839


🌐 Translation status

UI's texts

Language Nb. missing translations Status
en-us Ø ✔️

FAQ's questions

Language Nb. missing translations Status
en-us 14 ⬇️
Check missing rulesges,international,2tonnes,déprime,défaut,services-publics,services-publics-majoritaires,effet-qualité,bio-2,régime,pas-de-voiture,autres-carburants,électricité-dite-verte,impact-numérique

You will find more information about the translation in the dedicated file.

@Clemog
Copy link
Contributor

Clemog commented Feb 15, 2023

J'ai également séparé le scripts yarn start afin d'éviter d'avoir des process qui tournent dans le bg. Qu'est-ce que vous en pensez @laem et @Clemog ?

J'ai l'impression que dans la version en prod actuellement, le process est bien stoppé au ctrl c pourtant ?

@EmileRolley EmileRolley marked this pull request as ready for review February 16, 2023 09:38
@laem
Copy link
Contributor

laem commented Feb 22, 2023

@EmileRolley J'ai fait un tour de l'app en mode test utilisateur complet. Ça me semble bon. J'ai vérifié dans la console les règles qui étaient chargée.

🔴 J'ai vérifié les personas : il y a des différences. Sont-elles dues à une désynchro de master et de cette branche ?

Par contre je n'ai pas testé la combinaison compression / région.

Donc

  • pas de différence pour les 9 personas, test à la main, avant de merger
  • tester les changements de région en mode opti

@EmileRolley
Copy link
Contributor Author

EmileRolley commented Feb 22, 2023

🔴 J'ai vérifié les personas : il y a des différences. Sont-elles dues à une désynchro de master et de cette branche ?

J'ai testé plusieurs fois et je n'ai pas trouvé de différences.. Tu peux me donner la liste de celles que tu as trouvées stp ?

EDIT: les différences étaient bien dues à une désynchronisation avec master sur la PR du modèle.

@EmileRolley EmileRolley merged commit dd456fa into master Feb 27, 2023
@EmileRolley EmileRolley deleted the opti-activation-from-master branch February 27, 2023 17:39
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.

Use the new compression tool to generate optimized models
3 participants