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
Aquila Protocol Decoder - OBD Device Support #4892
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks pretty good. Added several minor comments mostly around code formatting and comments.
src/test/java/org/traccar/protocol/AquilaProtocolDecoderTest.java
Outdated
Show resolved
Hide resolved
@@ -49,6 +49,9 @@ public void testDecodeA() throws Exception { | |||
verifyPosition(decoder, text( | |||
"$$CLIENT_1WP,141216511,3,12.963212,77.533989,150908164041,V,31,0,0,0,8,0,0,0,0,0,0,0,1,0,1,0,0,1,0,0,0,0,1,0,0,0,0,0,0,0,0,0,0,1,1,1,0,*2A")); | |||
|
|||
// OBD Device | |||
verifyPosition(decoder, text( | |||
"$$CLIENT_1NS,866897057843486,1,13.144755,80.221474,220712112403,A,26,13,907034,263,15,0.830000,0,0,0,526336,13437,4004,3,21,1|010C:06410C14080000|010D:06410D03000000|0104:064104ED000000|010F:06410F6F000000|011F:06411F00050000|0121:06412100000000|0130:0641300D000000|0142:06414233CD0000|0131:06413103F30000|0143:067F43FFFF0000|015C:067F5CFF000000|012F:067F2FFF000000|0110:0641100C470000|0102:067F02FFFF0000|*5A")); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another small comment - please check an actual attribute to verify that OBD was decoded. And add a new line below.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed the comment and added a new line below.
I'm not sure how to check an actual attribute to verify that OBD was decoded.
Please point me to an example within another protocol from which I can know how to do this. Thank you.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can look at other decoder tests for examples. Like this one:
Hi,
This PR targets adding support for OBD device model using the aquila protocol.
Device seller website page:
https://www.itriangle.in/aquila-hardware-board-diagnostics-obd-ii.html
Please find attached the protocol manual for the device.
OBD 2_Protocol Manual.pdf
If this PR is accepted, please add a new device to the traccar.org supported devices list:
Device - "OBD II - 2G" | Protocol - aquila | Port - 5089
Hope this addition helps other traccar users and I thank the creators and maintainers of this wonderful project for making it open source.