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

Adding references to the new NMEA2000_teensy in the READMEs #1

Closed

Conversation

sarfata
Copy link
Contributor

@sarfata sarfata commented Nov 17, 2015

Thanks @ttlappalainen for this amazing NMEA2000 library. It seems to work very well with a Teensy board.

I have it connected to a NMEA2000 network with a Raymarine i70 and an Airmar DST800. Communication works from the devices to the board and from the board to the i70.

I have noticed that there seemed to be a problem in the conversion of doubles (the speed displayed on the i70 seems to be double the speed I set in the code). I am not sure if that is teensy specific or not. Will continue to look into this.

@ttlappalainen
Copy link
Owner

Hi,

Have you noticed that for library you have to use m/s for setting the
speed. So if you set 10 to library, it means 10 m/s, but Raymarine may
show it in knots and 10 m/s is about 20 knots.

If you need to to anything with NMEA2000<->NMEA0183, I have also library
for NMEA0183 for that, which I have not yet published. I have my own
NMEA0183->NMEA2000 converter.

Timo

On 17.11.2015 11:01, Thomas Sarlandie wrote:

Thanks @ttlappalainen https://github.com/ttlappalainen for this
amazing NMEA2000 library. It seems to work very well with a Teensy board.

I have it connected to a NMEA2000 network with a Raymarine i70 and an
Airmar DST800. Communication works from the devices to the board and
from the board to the i70.

I have noticed that there seemed to be a problem in the conversion of
doubles (the speed displayed on the i70 seems to be double the speed I
set in the code). I am not sure if that is teensy specific or not.
Will continue to look into this.


    You can view, comment on, or merge this pull request online at:

#1

    Commit Summary

* Adding references to the new NMEA2000_teensy in the READMEs


    File Changes

* *M* README.adoc
  <https://github.com/ttlappalainen/NMEA2000/pull/1/files#diff-0> (3)
* *M* README.md
  <https://github.com/ttlappalainen/NMEA2000/pull/1/files#diff-1> (2)


    Patch Links:

* https://github.com/ttlappalainen/NMEA2000/pull/1.patch
* https://github.com/ttlappalainen/NMEA2000/pull/1.diff


Reply to this email directly or view it on GitHub
#1.

@sarfata
Copy link
Contributor Author

sarfata commented Nov 17, 2015

Thanks, I will check the units. Did not think about that last night. I would love to see the NMEA0183 library, conversion from one to the other is what I am working on at the moment.

@ttlappalainen
Copy link
Owner

Hi,

I published my NMEA0183 library and made example NMEA0183ToN2k. This is
actually part of the code I use in my device on boat.

Please use my direct email timo.lappalainen@kave.fi.

Timo

On 17.11.2015 21:07, Thomas Sarlandie wrote:

Thanks, I will check the units. Did not think about that last night. I
would love to see the NMEA0183 library, conversion from one to the
other is what I am working on at the moment.


Reply to this email directly or view it on GitHub
#1 (comment).

@ttlappalainen
Copy link
Owner

Due to my inexperience with GitHub, I made these changes manually to my master branch.

ttlappalainen pushed a commit that referenced this pull request Mar 13, 2016
ttlappalainen pushed a commit that referenced this pull request Mar 23, 2016
Merge pull request #6 from adwuk/master
ttlappalainen pushed a commit that referenced this pull request Jun 25, 2016
ttlappalainen pushed a commit that referenced this pull request Mar 18, 2017
Refreshing from TTLap master
ttlappalainen pushed a commit that referenced this pull request Jun 5, 2017
Update my fork to latest upstream
ttlappalainen pushed a commit that referenced this pull request Nov 8, 2017
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.

None yet

2 participants