Skip to content

Conversation

@GwnDaan
Copy link
Member

@GwnDaan GwnDaan commented Jan 7, 2025

Describe your changes

Adds the VSCAN plugin (made by VSCOM) that works with several devices. It utilizes CANUSB/SLCAN interface over ASCII protocol.

It requires you to set the 'VSCAN_API_DIR' environment variable to the API SDK directory that contains the .h, .lib files. Furthermore, you need to put vs_can_api.dll in the same directory as the executable.

How has this been tested?

Tested this with a WiCAN-USB adapter over the TCP connection. The adapter was set to AP mode and I directly connected to it over wifi. No problems uploading the seeder example, nor any issues going through the example. Note that the busload here is fairly low here, as only the VT, TC and seeder example were running in a controlled environment.

I don't recommend to use any adapter over a wireless connection in a production environment, but for testing or datalogging it can be useful.

std::shared_ptr<isobus::CANHardwarePlugin> canDriver = std::make_shared<isobus::VSCANPlugin>("192.168.80.1:23");

@GwnDaan GwnDaan changed the title [HWadd VSCAN interface [HW]: add VSCAN hardware interface Jan 7, 2025
@GwnDaan GwnDaan requested a review from ad3154 January 7, 2025 21:25
@GwnDaan GwnDaan self-assigned this Jan 7, 2025
@GwnDaan GwnDaan added os: windows Related to usage on Windows os: linux Related to usage on Linux labels Jan 7, 2025
@GwnDaan GwnDaan force-pushed the daan/vscan-plugin branch 2 times, most recently from 9de44d7 to 4bd33f5 Compare January 8, 2025 11:29
ad3154
ad3154 previously approved these changes Jan 10, 2025
requires you to set the 'VSCAN_API_DIR' environment variable to the API SDK directory that contains the `.h`, `.lib` files. Furthermore, you need to put `vs_can_api.dll` in the same directory as the executable.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

os: linux Related to usage on Linux os: windows Related to usage on Windows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants