Skip to content

Conversation

RaulTrombin
Copy link
Member

@RaulTrombin RaulTrombin commented Oct 10, 2024

@patrickelectric can you forward to omniscan mainteners ? Nick?

Need to test on a real device:
cargo run --example omniscan450 -- --serial-port /dev/ttyUSB0

Need:

@RaulTrombin
Copy link
Member Author

RaulTrombin commented Oct 10, 2024

@Williangalvani
can you download the example on artifacts and test it?!
./omniscan450 --serial-port /dev/ttyUSB0

@patrickelectric
Copy link
Member

ping @Williangalvani

@patrickelectric
Copy link
Member

ping @NickNothom

@RaulTrombin RaulTrombin force-pushed the add_omniscan_example branch from d7c973e to ef4efc8 Compare April 16, 2025 20:00
@RaulTrombin
Copy link
Member Author

@patrickelectric @NickNothom @Williangalvani

seems that SonarView extension needs to be disabled for testing it.

UDP test:

+ ssh -t will-boat 'cd /home/pi/deploy && ls && ./omniscan450 --udp-address 192.168.2.92 --udp-port 51200'
omniscan450
Parsing user provided values and creating port...
Using UDP address: 192.168.2.92
Creating your Omniscan device
Device information: DeviceInformationStruct { device_type: 101, device_revision: 2, firmware_version_major: 1, firmware_version_minor: 3, firmware_version_patch: 0, reserved: 0 }
Connection to 10.147.20.119 closed.

TCP test:

+ rsync ./target/armv7-unknown-linux-gnueabihf/release/examples/omniscan450 will-boat:/home/pi/deploy
+ ssh -t will-boat 'cd /home/pi/deploy && ls && ./omniscan450 --tcp-address 192.168.2.92 --tcp-port 51200'
omniscan450
Parsing user provided values and creating port...
Using TCP address: 192.168.2.92
Successfully connected to TCP server
Creating your Omniscan device
Device information: DeviceInformationStruct { device_type: 101, device_revision: 2, firmware_version_major: 1, firmware_version_minor: 3, firmware_version_patch: 0, reserved: 0 }
Connection to 10.147.20.119 closed.

Copy link
Member

@Williangalvani Williangalvani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, but I'd not a rust guy, I'd wait for @patrickelectric 's review


let device_information = omniscan450.device_information().await?;

println!("Device information: {device_information:?}");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it possible for us to have more messages ?

@patrickelectric patrickelectric merged commit 3d2827b into bluerobotics:master Apr 25, 2025
5 checks passed
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.

3 participants