search

uqbar-project / wollok-ts-cli

Wollok Command Line Interface
GNU General Public License v3.0
9 stars 6 forks source link

Revisar los if(debug) time timeEnd #32

Open asanzo opened 2 years ago

asanzo commented 2 years ago

Hay que pegarle una revisada a los logs de profiling como estos:

https://github.com/uqbar-project/wollok-ts-cli/blob/47b6bed1ea9a8d1b4fc3ed25e2899aa3f9981059/src/commands/run.ts#L30-L32

https://github.com/uqbar-project/wollok-ts-cli/blob/47b6bed1ea9a8d1b4fc3ed25e2899aa3f9981059/src/utils.ts#L15-L29

Quizás necesitemos otra herramienta de logging que me permita hacer esos profilings en lugar de usarla como un flag y nada más.

PalumboN commented 8 months ago

@fdodino esto se cierra con el logger que tenemos, no?

fdodino commented 8 months ago

@PalumboN , maso, podríamos

  1. eliminar los if (debug) pero además
  2. hay que ver qué estrategia adoptamos con el logger. En Wollok LSP IDE guardamos el tiempo de buildeo que excede los xxx milisegundos (por defecto 100). Pero en wollok-ts-cli no estamos logueando lo que tarda, con lo cual cuando hacemos un run o levantamos un REPL esa info no está.
PalumboN commented 8 months ago

Qué significa eliminar los if (debug)?

fdodino commented 8 months ago

Y... ya no necesitaríamos tener un time + timeEnd con el debug, habría que cambiar la estrategia para guardar el tiempo solamente y llamar al logger por otro.