Skip to content

Conversation

juanjose58689
Copy link

No description provided.

@AlexFBP
Copy link
Collaborator

AlexFBP commented Mar 20, 2019

En general lo veo bien, solamente hacer unas correcciones de indentacion que iré colocando por aca 👍

@AlexFBP
Copy link
Collaborator

AlexFBP commented Mar 20, 2019

Antes de la línea 167 (antes de SendMeasures(PY_UNCONFIRMED);) podrían ir colocando un código similar al que dejaron en el Test. Por otro lado, el test podría estar en un archivo aparte en la carpeta de ejemplos, pero eso ultimo no es estrictamente necesario...

printf("%f\n%f\n",aa,bb);
// phlen=sprintf(phrase,"Ozono:%f\r\nCalidad del Aire:%f\r\nMonoxido de Carbono:%f\r\n\n\n",aa,bb,cc);
// SerialDevice_SendData(phrase,phlen);
// ms100(10);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quedaría faltando la parte de enviar esos datos por LoRaWAN, un ejemplo para no complicarse tanto la vida sería extendiendo la funcionalidad de los archivos pylatex.[ch], y eso de la mano con las modificaciones que haya lugar en el decoder, pero para ser una prueba esta bien 👍

@AlexFBP
Copy link
Collaborator

AlexFBP commented Mar 31, 2019

@juanjose58689 como le(s) fue integrando la librería al ejemplo principal? No han publicado (git push) mas commits a esta rama...

@juanjose58689
Copy link
Author

juanjose58689 commented Mar 31, 2019 via email

@AlexFBP AlexFBP self-requested a review May 7, 2019 05:10

#ifndef _XTAL_FREQ
#define _XTAL_FREQ 8000
#endif
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@juanjose58689 La directiva XTAL_FREQ se encuentra ya definida en pylatex.h, por lo que las líneas 11-13 se pueden eliminar, y al comienzo de este archivo colocar un `#Include "nucleoPIC.h", como en el #5 ... #5 (comment)

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(De hecho mas bien no al comienzo de este archivo ADC.h sino del ADC.c)

@AlexFBP
Copy link
Collaborator

AlexFBP commented May 7, 2019

@juanjose58689 si ha(n) tenido algun problema al momento de compilar, comentenlo por acá please. Con respecto al comentario que les dejé del #include ... @camilop98 y/o @alejacr1997 puedan explicarles un poco mejor eso, pero en resumen _XTAL_FREQ ya está definida como 8000000 en nucleoPIC.h, solamente es consumirla donde sea necesario

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

Successfully merging this pull request may close these issues.

2 participants