Conversation
alutenberg
left a comment
alutenberg
left a comment
There was a problem hiding this comment.
Está bastante bien tu entrega, de todas formas de dejé varios comentarios para que por favor revises.
README.md
Outdated
| El sistema controlará el suministro y la producción de agua ultrapura a través de un sistema de tratamiento mediante ósmosis inversa. | ||
|
|
||
| Entrega TP1 | ||
| - Encender y apagar el sistema, en este caso ya se habia implementado maquina de estado para ese control por que se evidenció problema de rebote y era muy dificil poder prender o apagar el sistema. el Led1 es el indicador de este servicio. |
There was a problem hiding this comment.
- Encender y apagar el sistema: en este caso ya se habia implementado
Van dos puntos luego de sistema.
README.md
Outdated
|
|
||
| Entrega TP1 | ||
| - Encender y apagar el sistema, en este caso ya se habia implementado maquina de estado para ese control por que se evidenció problema de rebote y era muy dificil poder prender o apagar el sistema. el Led1 es el indicador de este servicio. | ||
| - Controlar las presiones de corte mínimas y máximas para el encendido de las bombas de abastecimiento principal, este item se esta emulando con un potenciometro y se realiza una conversión sencilla que me permite obtener la presión de entrada en PSI y prender o apagar la bomba P1 según la presiones altas y bajas configuradas para apagar el sistema, el monitoreo se realiza cada 1.5 segundos, evitando ser bloqueante. |
There was a problem hiding this comment.
"bombas de abastecimiento principal: este..."
README.md
Outdated
|
|
||
| Entrega TP1 | ||
| - Encender y apagar el sistema, en este caso ya se habia implementado maquina de estado para ese control por que se evidenció problema de rebote y era muy dificil poder prender o apagar el sistema. el Led1 es el indicador de este servicio. | ||
| - Controlar las presiones de corte mínimas y máximas para el encendido de las bombas de abastecimiento principal, este item se esta emulando con un potenciometro y se realiza una conversión sencilla que me permite obtener la presión de entrada en PSI y prender o apagar la bomba P1 según la presiones altas y bajas configuradas para apagar el sistema, el monitoreo se realiza cada 1.5 segundos, evitando ser bloqueante. |
There was a problem hiding this comment.
No me queda claro por qué al monitorear cada 1.5 se evita que sea bloqueante. Digo, lo que importa no es cada cuánto tiempo se puede bloquear, sino una vez que llega a la situación bloqueante, cuánto tiempo se queda sin salir de la situación bloqueante. Todo esto siempre en relación con la dinámica del sistema: si el sistema requiere demorar de décimas de milisegundos, entonces un bloquedo de cientos de milisegundos, por ejemplo, es una barbaridad.
There was a problem hiding this comment.
Ah, más abajo encontré la parte del código en la que haces que no sea tan molesto ni tan bloqueante el delay de 1,5 segundos, al separarlo en delays más pequeños.
README.md
Outdated
| Entrega TP1 | ||
| - Encender y apagar el sistema, en este caso ya se habia implementado maquina de estado para ese control por que se evidenció problema de rebote y era muy dificil poder prender o apagar el sistema. el Led1 es el indicador de este servicio. | ||
| - Controlar las presiones de corte mínimas y máximas para el encendido de las bombas de abastecimiento principal, este item se esta emulando con un potenciometro y se realiza una conversión sencilla que me permite obtener la presión de entrada en PSI y prender o apagar la bomba P1 según la presiones altas y bajas configuradas para apagar el sistema, el monitoreo se realiza cada 1.5 segundos, evitando ser bloqueante. | ||
| - Ajustar la frecuencia de funcionamiento de la bomba que proporciona presión a las membranas de filtración. Se ha implementado una maquina de estado para el manejo de un teclado 4x3, todavia no se ha implementado. |
README.md
Outdated
| - Encender y apagar el sistema, en este caso ya se habia implementado maquina de estado para ese control por que se evidenció problema de rebote y era muy dificil poder prender o apagar el sistema. el Led1 es el indicador de este servicio. | ||
| - Controlar las presiones de corte mínimas y máximas para el encendido de las bombas de abastecimiento principal, este item se esta emulando con un potenciometro y se realiza una conversión sencilla que me permite obtener la presión de entrada en PSI y prender o apagar la bomba P1 según la presiones altas y bajas configuradas para apagar el sistema, el monitoreo se realiza cada 1.5 segundos, evitando ser bloqueante. | ||
| - Ajustar la frecuencia de funcionamiento de la bomba que proporciona presión a las membranas de filtración. Se ha implementado una maquina de estado para el manejo de un teclado 4x3, todavia no se ha implementado. | ||
| - Envio por UART, de momento se esta enviando cada 3 segundos el estado del sistema y la presión actual del sistema. |
There was a problem hiding this comment.
envio por UART: de momento se e
Y además, ojo que "envio" debe llevar tilde en la "i", o sea, Envío.
main.cpp
Outdated
| void inputsInit(); | ||
| void outputsInit(); | ||
|
|
||
| void statusSistem(); |
| statusLed(); | ||
| checkpressure(); | ||
| sendData(); | ||
| HAL_Delay(TIME_INCREMENT_MS); |
There was a problem hiding this comment.
Dónde está definida esta función "HAL_Delay".
main.cpp
Outdated
|
|
||
| void outputsInit() | ||
| { | ||
| ledSistem = 0; |
There was a problem hiding this comment.
Buscar todas las instancias de "Sistem" y cambiarlas por "System", o bien por "Sistema"
| } | ||
| } | ||
|
|
||
| bool debounceButtonUpdate() |
There was a problem hiding this comment.
Fijate que acá estás usando una Máquina de Estado, y se supone que en el TP2 recién se incorpora el tema de FSM, pero sin embargo si lo sabes usar lo podes usar.
main.cpp
Outdated
| else{ | ||
| int valuePressureM1 = readPressureM1(); | ||
|
|
||
| if (valuePressureM1 <= 5) { |
There was a problem hiding this comment.
No se entiende bien de dónde salen estos valores mágicos, como 5, 6 y 75. Deberían estar en la zona de definiciones.
2 participants
No description provided.