Skip to content

Feature/sec arch#96

Open
caiussouza wants to merge 37 commits into
developfrom
feature/sec_arch
Open

Feature/sec arch#96
caiussouza wants to merge 37 commits into
developfrom
feature/sec_arch

Conversation

@caiussouza

Copy link
Copy Markdown

Implementation of a security architecture related to dynamic controls, specifically the torque vectoring. It performs cross validations between IMU x brake status and IMU x inductive speed sensors.

@caiussouza caiussouza requested review from FelTell and dmrohrs March 17, 2023 18:34
@caiussouza

Copy link
Copy Markdown
Author

Estou trabalhando para corrigir os problemas de clang-tidy (relacionados aos warnings de compilação)

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ clang-tidy found issue(s) with the introduced code (1/1)

Comment thread Core/Src/dynamic_controls/security_architecture.c Outdated
Comment thread Core/Src/dynamic_controls/security_architecture.c Outdated
Comment thread Core/Src/dynamic_controls/security_architecture.c Outdated
Comment thread Core/Src/dynamic_controls/security_architecture.c Outdated
Comment thread Core/Src/dynamic_controls/security_architecture.c Outdated
Comment thread Core/Src/dynamic_controls/security_architecture.c Outdated
Comment thread Core/Src/dynamic_controls/security_architecture.c Outdated
Comment thread Core/Src/dynamic_controls/security_architecture.c Outdated
Comment thread Core/Inc/util/global_definitions.h Outdated
Comment thread Core/Src/dynamic_controls/security_architecture.c
Comment thread Core/Inc/util/global_definitions.h Outdated
@dmrohrs dmrohrs requested a review from FelTell March 27, 2023 22:11
@FelTell

FelTell commented Apr 3, 2023

Copy link
Copy Markdown
Contributor

Faça merge com a branch da #97 e corrija o que o David pediu que ai eu olho denovo

@caiussouza

Copy link
Copy Markdown
Author

@FelTell @dmrohrs fiz as correções e tbm o merge com a develop(após o PR da wrong-names[...]). Tive alguns conflitos qnd dei merge, mas já resolvi. Apesar disso, pelo número de conflitos que aconteceu, gostaria q vcs prestassem atenção principalmente no main.c pra ver se cometi algum erro na hora de recriar os itens no .ioc

@dmrohrs

dmrohrs commented Apr 13, 2023

Copy link
Copy Markdown
Contributor

@FelTell Fiquei na dúvida com relação às mudanças dos arquivos de sistema. Vc sabe um jeito simples de identificar o que é proveniente da atualizações do CubeMX? De resto fechei o PR com missclick

@dmrohrs dmrohrs closed this Apr 13, 2023
@dmrohrs dmrohrs reopened this Apr 13, 2023
Comment thread Core/Inc/CAN/CAN_IDs.h Outdated
Comment thread Core/Inc/util/global_definitions.h Outdated
Comment thread Core/Inc/util/CMSIS_extra/global_variables_handler.h Outdated
Comment thread Core/Inc/CAN/CAN_IDs.h Outdated
Comment thread Core/Src/CAN/inverter_can_monitor.c
Comment thread Core/Src/dynamic_controls/security_architecture.c Outdated
Comment thread Core/Src/sensors/sensor_data_processing.c Outdated
Comment thread Core/Src/sensors/sensor_data_processing.c Outdated
Comment thread Core/Src/sensors/sensor_data_processing.c Outdated
Comment thread Core/Src/sensors/sensor_data_processing.c Outdated
@caiussouza caiussouza requested a review from dmrohrs May 25, 2023 19:18
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.

3 participants