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

Sacar de aquí el módulo de lectura, al menos de lectura directa. #21

Open
JJ opened this issue Apr 1, 2020 · 2 comments
Open

Sacar de aquí el módulo de lectura, al menos de lectura directa. #21

JJ opened this issue Apr 1, 2020 · 2 comments

Comments

@JJ
Copy link
Contributor

JJ commented Apr 1, 2020

Es lo que está fallando, aparentemente porque no hay un Arduino conectado. Primero, no se puede testear así, confiando que haya un chisme hardware conectado. Segundo, hace que perdamos un poco de foco. De la misma forma que el firmware, es mejor que vaya a otro repositorio y dejemos aquí solamente el módulo de lectura (simulado) y el UI.
En todo caso, voy a ver cómo está organizada la clase a ver si quizás lo que haya que hacer es dividirla.

@crisconru
Copy link
Member

Gracias @JJ yo de hecho no me gusta tener todo en un repo, me gustaría tener uno para cada cosa:

  • Documentación en uno.
  • Capa de comunicación USB en otro.
  • Traductores en otro.
  • GUI en otro
  • Etc.

Pero me daba cosa distribuirlo no vaya ser que fuera un caos. Pero es realmente lo ideal, y así poder tener los issues también de cada cosa independiente y ordenada. ¿Te importaría crearlos a ti Juan? Estoy liado tomando y documentando los requisitos del respirador (vamos a salir con el titulo de anestesísticas de aquí jeje).

@JJ
Copy link
Contributor Author

JJ commented Apr 1, 2020

Tampoco puedo asignar labels. Pero en todo caso este es el principal ahora. Hay que sacar todo esto de aquí para poder centrarnos en el GUI y en la lectura y procesamiento.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants