-
Notifications
You must be signed in to change notification settings - Fork 73
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WiFi #26
Comments
That would require quite some changes. Could be possible ofcourse, you just need to swap out the bluetooth-serial-connection with a wifi kind of module. |
I was thinking of switching out bluetooth with this. https://github.com/monteslu/udp-serial Not sure if this package works, I've only got a successful connection to my elm327. Haven't been able to read or write to the port. |
I've forked a version that uses TCP - I've tested it with obdsim so far. Seems to work fine. Work in progress. https://github.com/hsccorp/node-bluetooth-tcp-obd and an example at https://github.com/hsccorp/obd-tcp-test My JS skills aren't up to snuff, so I'm sure I've done things that are not ideal. Happy to accept PRs and happy to get this merged with @EricSmekens library if he'd like - but I have no way to test if my changes affected the BT part (plus it will need better cleanup) |
Cool, I will try to have a look soon. Would be cool to merge this, and make a generic module that could work with bluetooth/tcp and serial. But time is an issue. |
@arjunroychowdhury Did you manage to test this on a simulator? Would be nice to know what you think of this. |
@EricSmekens, yes. I'm using it with the TCP fork of obdsim and a real mobile app |
Can this be adapted to the wifi serial interface? 192.168.0.10:35000
The text was updated successfully, but these errors were encountered: