[PR] Added simulator nodes #14

Merged
vincent merged 14 commits from 3-odometry/accel-velocity-sensor-sim into 3-odometry/master 2025-11-28 09:35:25 +01:00
Member

Added:

  • Nodes
  • Tests
  • Documentation
Added: - Nodes - Tests - Documentation
vincent added 12 commits 2025-11-26 19:22:34 +01:00
vincent requested review from inholland 2025-11-26 19:22:39 +01:00
vincent requested review from wessel 2025-11-26 19:23:05 +01:00
Owner

Misschien alleen nog deceleration toevoegen? Dat je bijv. van 5 naar 0 naar -5 naar 0 gaat en dat repeat? Gewoon een idee, geen eis ;}

Misschien alleen nog deceleration toevoegen? Dat je bijv. van 5 naar 0 naar -5 naar 0 gaat en dat repeat? Gewoon een idee, geen eis ;}
wessel approved these changes 2025-11-28 07:43:15 +01:00
wessel left a comment
Owner

Paar kleine styling opmerkingen, ziet er voor de rest goed uit. Wat mij betreft kan je het ook laten want ik vraag me af of Tilmann er uberhaupt naar kijkt :}

Paar kleine styling opmerkingen, ziet er voor de rest goed uit. Wat mij betreft kan je het ook laten want ik vraag me af of Tilmann er uberhaupt naar kijkt :}
@@ -0,0 +1,18 @@
#include "rclcpp/rclcpp.hpp"
Owner

Bestandsnamen zijn nu wel snake_case, de rest van het project is UpperCamelCase

Bestandsnamen zijn nu wel `snake_case`, de rest van het project is `UpperCamelCase`
vincent marked this conversation as resolved
@@ -0,0 +27,4 @@
std::unique_ptr<Simulator> simulator_;
void publish_wheel_data();
Owner

Trailing newlines

Trailing newlines
vincent marked this conversation as resolved
@@ -0,0 +2,4 @@
#include <algorithm>
namespace assignments::three
{
Owner

Formatting

Formatting
@@ -0,0 +5,4 @@
#include <map>
namespace assignments::three
{
Owner

Formatting

Formatting
vincent marked this conversation as resolved
@@ -0,0 +36,4 @@
void load_intervals(rclcpp::Node* node, const std::vector<std::string>& objects);
double compute_value(double t, const IntervalConfig& interval);
std::map<std::string, std::vector<IntervalConfig>> object_intervals_;
Owner

Variables bij elkaar boven de functies

Variables bij elkaar boven de functies
vincent marked this conversation as resolved
@@ -0,0 +251,4 @@
EXPECT_DOUBLE_EQ(sim.get_object_value("test_channel", 25.0), 2.0); // Third interval should not exist, holds second
}
int main(int argc, char** argv) {
Owner

Overbodig chatstukje ;)

Overbodig chatstukje ;)
vincent marked this conversation as resolved
wessel added the Kind/Feature
Priority
High
2
Assignment 3
labels 2025-11-28 07:44:26 +01:00
wessel added a new dependency 2025-11-28 08:56:12 +01:00
Author
Member

deceleration kan je via opt file doen

deceleration kan je via opt file doen
Owner

deceleration kan je via opt file doen

Ohh top, excuses

> deceleration kan je via opt file doen Ohh top, excuses
vincent added 2 commits 2025-11-28 09:34:51 +01:00
vincent merged commit df9bafef0e into 3-odometry/master 2025-11-28 09:35:25 +01:00
Sign in to join this conversation.