Merge pull request #9 from ProfFan/fan/pdoa

Fixed non-default features usage

Author: ProfFan

.github/workflows/rust.yml

@@ -28,3 +28,11 @@ jobs:
     - uses: actions/checkout@v4
     - name: Run clippy
       run: cargo clippy --all -- -D warnings
+
+  # No Default Features
+  no-default-features:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v4
+    - name: Run tests with no-default-features
+      run: cargo test --no-default-features --verbose

Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 name = "dw3000-ng"
 description = "A modernized driver for DW3000 Ultra Wide Band module"
-version = "0.7.0"
+version = "0.7.1"
 authors = [
     "Fan Jiang <i@fanjiang.me>",
     "Clement Pene <clement.pene@sii.fr>",
@@ -23,7 +23,7 @@ smoltcp = { version = "0.11", default-features = false, features = [
 ] }
 nb = "1.0"
 fixed = "1.7"
-defmt = "0.3"
+defmt = { version = "0.3", optional = true }
 num-traits = { version = "0.2.18", default-features = false, features = ["libm"], optional = true }
 
 [dev-dependencies]
@@ -42,5 +42,5 @@ default-features = false
 default = ["async", "defmt", "num-traits"]
 std = ["serde/std", "num_enum/std"]
 async = []
-defmt = []
+defmt = ["dep:defmt"]
 num-traits = ["dep:num-traits"]

README.md

@@ -13,7 +13,7 @@ A modernized driver for the Decawave [DW3000] UWB transceiver, written in the [R
 Both RTT methods (single and double sided) are working and giving good positioning values.
 No implementation of PDoA or AoA.
 
-Compared to the old `dw3000` crate we fixed the GPIOs and LEDs, also got rid of the old unmaintained ieee802154 crate and replaced it with smoltcp.
+Compared to the old `dw3000` crate we fixed the GPIOs and LEDs, also got rid of the old unmaintained ieee802154 crate and replaced it with `smoltcp`.
 
 We mainly test on the ESP32 platform with `embassy` async framework.
 

src/hl/error.rs

@@ -185,6 +185,7 @@ mod test {
 
     use embedded_hal_mock::eh1::spi::Mock as SpiMock;
 
+    #[cfg(feature = "defmt")]
     #[test]
     fn test_defmt() {
         let error = Error::<SpiMock<u8>>::BufferTooSmall { required_len: 42 };

src/hl/receiving.rs

@@ -38,7 +38,8 @@ pub struct Message<'l> {
 }
 
 /// A struct representing the quality of the received message.
-#[derive(Debug, Format, serde::Serialize, serde::Deserialize)]
+#[cfg_attr(feature = "defmt", derive(Format))]
+#[derive(Debug, serde::Serialize, serde::Deserialize)]
 pub struct RxQuality {
     /// The confidence that there was Line Of Sight between the sender and the
     /// receiver.
@@ -426,6 +427,11 @@ where
             - a)
     }
 
+    #[cfg(not(feature = "num-traits"))]
+    fn get_first_path_signal_power(&mut self) -> Result<f32, Error<SPI>> {
+        Ok(0.0)
+    }
+
     #[allow(clippy::type_complexity)]
     /// Finishes receiving and returns to the `Ready` state
     ///

src/lib.rs

@@ -4,21 +4,11 @@
 //! require a higher degree of flexibility, you can use the
 //! [register-level interface] instead.
 //!
-//! We used the crate [`dw1000`] developped for the DW1000 module and changed
-//! the registers access and spi functions, added fast command and implemented
-//! some high level functions.
-//!
-//! We tried a first positionning exemple using RTT methode.
-//! Lot of work still need to be added like the use of PDoA or AoA.
-//!
-//! These examples uses a NUCLEO STM32F103RB
-//!
 //! This driver is built on top of [`embedded-hal`], which means it is portable
 //! and can be used on any platform that implements the `embedded-hal` API.
 //!
 //! [high-level interface]: hl/index.html
 //! [register-level interface]: ll/index.html
-//! [`dw1000`]: https://crates.io/crates/dw1000
 //! [`embedded-hal`]: https://crates.io/crates/embedded-hal
 #![cfg_attr(not(any(test, feature = "std")), no_main)]
 #![cfg_attr(not(any(test, feature = "std")), no_std)]