Open
Conversation
JorgePerC
requested changes
Jan 10, 2025
Contributor
JorgePerC
left a comment
There was a problem hiding this comment.
Recuerden que esto eventualmente se volverá una task.
Vayan definiendo las queues para leer los datos de las otras tasks
src/chassisMove.cpp
Outdated
| //se llaman los valores dados por el acelerometro de la imu y por las velocidades de los motores. | ||
| void chassisMove::LecturaDeDatos(float x1, float y1,float z1 , float leftFrontSpeed, float rightFrontSpeed, float rightBackSpeed, float leftBackSpeed){ | ||
|
|
||
| float x1g = x1 / 16384.0; // Conversión a G (para ±2g) esto solo los valores del acelerometro(tengo duda si se tiene que convertir) |
Contributor
There was a problem hiding this comment.
Está bien la conversión.
Crees poderle poner un comentario del valor de ese número...? Es por la resolución? alguna constante de la gravedad? Y si se puede, link a la referencia
JorgePerC
approved these changes
Jan 13, 2025
Contributor
JorgePerC
left a comment
There was a problem hiding this comment.
Está super! Ahora sólo ponte de acuerdo con Diego, sobre cuál va a ser el struct /queue por el cual van a mandar los datos
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hola, hice unas modificaciones para lo de imu. :)