[Bug]: Unused method warning in `SimulationBldcDriver`

[gelanchez]

Bug Description

The compiler is reporting a dead_code warning in the simulation driver for BLDC motors. The method throttle_to_pwm is defined but never called within the current implementation of the simulation logic.

Location: horus_library/drivers/bldc/simulation.rs

Steps to Reproduce

1. Create a new project.
2. Build and run.

Expected Behavior

No warnings.

Actual Behavior

warning: method throttle_to_pwm is never used
--> /home/jose/git/PUBLIC/horus/horus_library/drivers/bldc/simulation.rs:69:8
|
24 | impl SimulationBldcDriver {
| ------------------------- method in this implementation
...
69 | fn throttle_to_pwm(&self, throttle: f64) -> u16 {
| ^^^^^^^^^^^^^^^
|
= note: #[warn(dead_code)] (part of #[warn(unused)]) on by default

HORUS Version

0.1.6

Operating System

Ubuntu 24.04

Rust Version

rustc 1.92.0

Language

Rust

[gelanchez]

I've just seen the unused method warning in SimulationBldcDriver is being addressed in dev with #[allow(dead_code)].

It might be worth considering making the method pub if it’s meant to be part of the public API, even if it's not used right now. Just a thought!