search

SignalK / signalk-autopilot

Apache License 2.0
16 stars 13 forks source link

Code change request #36

Open PatrickB100 opened 2 months ago

PatrickB100 commented 2 months ago

Hi All,

I had to make some minor changed to get the integration to work with the Miniplex 3 and a Seatalk 1 connected autopilot. Could those changed be reviewed and corrected?

raymarinest.js

Change code from

function toSentence(parts) {
  var base = parts.join(',');
  return base + computeChecksum(base);
}

Change code to work with EV-1 on ST1

function toSentence(parts) {
  var base = parts.join(',');
  return base;
}

Change code from 

    var sentence = toSentence([
      '$PSMDST',
      datagram

Change code to work with new ST sentences V2.0

    var sentence = toSentence([
      '$PSMDST,C',
      datagram
sbender9 commented 2 months ago

I wonder if this would break things for others?

If so, then an option or new file should be added.

And please do a PR, I have no way to test this.