Skip to content

Add Radshid 2G/4G protocol support #5583

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

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

0x7a657573
Copy link

This branch was opened last year but I didn’t have time to finish it until now. I finally revisited the code, fixed several wording / formatting issues in the docs and comments, and made sure everything builds cleanly against the current
master.

Radshid is a company that make 2G/4G trackers, I added the radshid protocol to traccar.
you can find Radshid protocol on #5235.

Comment on lines +147 to +152
byte ioStatus = packetData.readByte(); // IO Status
byte gpsStatus = packetData.readByte(); // GPS Status
long latitude = packetData.readInt(); // Latitude
long longitude = packetData.readInt(); // Longitude
int altitude = packetData.readShort(); //Altitude
int bearing = packetData.readShort(); //Bearing
Copy link
Member

Choose a reason for hiding this comment

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

Remove unnecessary variables and comments.

import java.util.LinkedList;
import java.util.List;

public class RadshidProtocolDecoder extends BaseProtocolDecoder {
Copy link
Member

Choose a reason for hiding this comment

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

You also have to add unit tests for the protocol.

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.

2 participants