search

tomasregna / WOW

WOW: WOW Observational Work, un programa dedicado a la creación de Backups, Reducción de imágenes de ciencia, Fotometría de apertura y generación de Tablas Fotométricas
GNU General Public License v3.0
2 stars 0 forks source link

Impresión por pantalla #17

Closed nguevara closed 4 years ago

nguevara commented 4 years ago

Esta seteado en todos lados que no imprima cosas por pantalla pero aún así lo hace. Como excepción logre que no lo haga para los hedit, pero para todo el resto (process y photometria) sí lo hace. También levanta varios warnings al estilo de "aca no pusiste el dtype" o cosas así que no está bueno que salgan. Me pareció que podíamos ver de que todo lo que sea "stdout" (osea salida estandar) que lo imprime por pantalla lo grabemos en un file tipo "logfile". Habría que guglear esto o consultar a yael. Como yapa, se podría agregar un mensaje después de cada paso exitoso indicando que lo hizo. Osea "fotometria ok" "reduccion ok" "ok boomer" ahre

tomasregna commented 4 years ago

Respecto a:

  1. La impresión de cosas por pantalla, podríamos correrlo por la terminal de Spyder en vez de hacerlo ejecutable, ya que ahí siempre se portó bien, con pocos mensajes de pantalla. Y de ser inevitables bueno, siempre está la filosofía programativa de: it's not a bug, it's a feature! y decir que lo de que te pida varios ENTER es un mecanismo de control que agregamos... :dancer:

  2. Los warnings, resulta medio sospechoso que no saltaran antes; más pruebas del traje de baño deberían aclarar algunas cuestiones.

  3. La salida estándar y el logfile, me parece que no es prioritario, lo importante ahora es que sea funcional, y una vez logramos tener algo estable, para la v2.0 podríamos considerar agregar la opción del logfile.

4.Los mensajes tras los pasos me va, son agregar simplemente unos cuantos print, así que adhiero!

nguevara commented 4 years ago
  1. oh god no. No me cabe correrlo por spider, pero si es lo que queda... ok :'(
  2. Los warnings si saltaban, pasa que los ignorabamos. Siempre saltaron.
  3. de una
  4. :+1:
nguevara commented 4 years ago

lo acabo de correr por spyder y efectivamente no pregunta nada. Corre de una. Pero sigue imprimendo por pantalla todo lo que hace, aunque le pedi que no..

tomasregna commented 4 years ago

Maldi tasea

1.Tenemos ambas opciones, la diferencia es si queremos poner ENTER o no, pero me da igual a mí.

2.Buen punto; probablemente ¿nos excedan?

nguevara commented 4 years ago

Resuelto