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

Create modules skeleton for new app structure #382

Merged
merged 2 commits into from
Jun 22, 2021

Conversation

cindyli-13
Copy link
Member

The goal is to create a new modularized structure for all rover apps. See confluence for details: https://wiki.uwaterloo.ca/display/UWRT/FW+Rover+Apps+Structure

@cindyli-13 cindyli-13 changed the title Create module skeletons for new app structure Create modules skeleton for new app structure Jun 17, 2021
@cindyli-13 cindyli-13 marked this pull request as ready for review June 18, 2021 01:49
@cindyli-13 cindyli-13 merged commit acd0b4d into app-structure-redesign Jun 22, 2021
@cindyli-13 cindyli-13 deleted the cindy/create-modules-skeleton branch June 22, 2021 16:09
cindyli-13 added a commit that referenced this pull request Oct 17, 2021
* Add modules and main.cpp skeleton

* Fix typos
cindyli-13 added a commit that referenced this pull request Jan 16, 2022
* Add modules and main.cpp skeleton

* Fix typos
cindyli-13 added a commit that referenced this pull request Jan 18, 2022
* Create modules skeleton for new app structure (#382)

* Add modules and main.cpp skeleton

* Fix typos

* Create watchdog module (#389)

* Watchdog module

* clang fix

* clang fix2

* test-watchdog

* Edited WatchdogWrapper.h

* Fixed pure virtual function errors

* fixed namespace error

* Fixed error2

* Cmake

* Cmake

* Cmake fix

* Top-level Cmake

* Changes made

* clang fix

* Update libs/utility/src/WatchdogWrapper.cpp

Co-authored-by: Cindy Li <[email protected]>

* Update test-apps/test-watchdog/src/main.cpp

Co-authored-by: Cindy Li <[email protected]>

* cmake

* Changed periodic function name

* watchdogwrapper fix

Co-authored-by: Cindy Li <[email protected]>

* Create PDB monitoring module (#392)

* PDB Module added

* CMake

* rename include

* Fixed Errors

* clang fix

* Changes made

* Made pins config private members

* clang

* fix folder structure

* fix common modules not found

* rebase master

* Address pr comments

Co-authored-by: jahnavithota2011 <[email protected]>
upadhyaydhruv pushed a commit that referenced this pull request Feb 4, 2022
* Add modules and main.cpp skeleton

* Fix typos
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.

1 participant