-
Notifications
You must be signed in to change notification settings - Fork 40
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
Conversation
✅ Deploy Preview for nosgestesclimat ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Report for the pull request #839🌐 Translation statusUI's texts
FAQ's questions
|
e6ab911
to
fe94f1d
Compare
@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
|
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. |
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).