Simulation Interface Draft Stubs

[colinli02]

Description

• Resolves CAN Sim Interface Stubs #16

Verification

• [ ]

Resources

[colinli02]

@hhenry01

Hey Henry, I have:

• resolved merge conflicts
• removed 'auto'
• removed RCLCPP_INFO() logging from terminal

Should be ready for merge

[hhenry01]

Make to follow ROS style guidelines, in particular for variables: http://wiki.ros.org/CppStyleGuide#Variables.

[hhenry01]

For every .cpp and .h file you've touched, open the command palette (ctrl+shift+p) and run format document. This should run automatically on manual save, but I'm assuming you have auto-save on.

[colinli02]

For every .cpp and .h file you've touched, open the command palette (ctrl+shift+p) and run format document. This should run automatically on manual save, but I'm assuming you have auto-save on.

I have auto-save off on vscode, I normally just click ctrl + s after a couple lines out of muscle memory, is the formatting not working / not expected what it should look like?

I've ran the formatter under command palette either way, with clangd format

[hhenry01]

For every .cpp and .h file you've touched, open the command palette (ctrl+shift+p) and run format document. This should run automatically on manual save, but I'm assuming you have auto-save on.

I have auto-save off on vscode, I normally just click ctrl + s after a couple lines out of muscle memory, is the formatting not working / not expected what it should look like?

I've ran the formatter under command palette either way, with clangd format

Formatting was broken and I didn't catch it because of one of my user settings. I'll fix it soon. Thanks for catching this.

Edit: Fixed in: UBCSailbot/sailbot_workspace#202

[hhenry01]

Make to follow ROS style guidelines, in particular for variables: http://wiki.ros.org/CppStyleGuide#Variables.

Take another look at this. Particularly for variable names.

[hhenry01]

There appears to be misunderstanding for how the CanSimIntf plugs in to the overall software flow alongside the regular CanTransceiverIntf. We'll need to talk about this more in depth.

[colinli02]

Make to follow ROS style guidelines, in particular for variables: http://wiki.ros.org/CppStyleGuide#Variables.

Take another look at this. Particularly for variable names.

I've changed all variables and member variable names to underscored and lower case