-
Notifications
You must be signed in to change notification settings - Fork 7
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
Fix #143 #145
Fix #143 #145
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #145 +/- ##
==========================================
- Coverage 84.35% 84.33% -0.03%
==========================================
Files 9 9
Lines 684 683 -1
Branches 162 162
==========================================
- Hits 577 576 -1
Misses 107 107 ☔ View full report in Codecov by Sentry. |
Hola, con este PR me puse a buscar donde se perdía mas tiempo wollok-ts-cli/src/commands/run.ts Line 255 in f6cf36c
Mirando mas a fondo, noté que acá se demoraba mucho, el tema es que no termino de entender que es ese round wollok-ts-cli/src/commands/extrasGame.ts Line 36 in f6cf36c
Eliminando esa línea el tiempo baja de 120ms a unos 20ms. Tendría que saber cual es la función de ese round para ver si se puede implementar directamente en ts |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alto laburo Manuel!!
Ahí te dejé un comentario, similar al del PR de TS.
Habría que ver cómo te dan los tiempos si hacemos el redondeo en TS en vez de en Wollok. Pero los número que tenés ahora da menos porque perdimos un feature.
Por otro lado, cómo hiciste para profilear y encontrar la línea que tarda? Es interesante
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
También en algún momento habría que:
- Testear todas estas funciones de este archivo
- Ver si conviene moverlas a TS
Este PR es para probar alternativas para mejorar la performance de wollok game