Skip to content
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

Code change request #36

Open
PatrickB100 opened this issue Jul 20, 2024 · 1 comment
Open

Code change request #36

PatrickB100 opened this issue Jul 20, 2024 · 1 comment

Comments

@PatrickB100
Copy link

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
Copy link
Member

sbender9 commented Jul 20, 2024

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.

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

No branches or pull requests

2 participants