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

updating nmea_proxy to support fowarding to a secondary destination like Avalon #44

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

webinage-tech
Copy link

@webinage-tech webinage-tech commented Jun 15, 2021

This pull request is provided to you by EffectiveTeam. It enables both Avalon and QTVlm routers to get NMEA data simultaneously from VRDashboard.

Without this enhancement, only 1 can receive data from VRDashboard ; on the other, boat position is painfully updated manually.

Carefull attention has been paid to not disturb current nmea_proxy.py behaviour.
In order to enable the new feature and forward data to Avalon, simply use the new "--fport <fowarding_port>" option and set accordingly this forwarding port on Avalon race setting as NMEA source.

Proper logging and error management has been included.
Code has been tested successfully in various configuration.

Copy link

@toxcct toxcct left a comment

Choose a reason for hiding this comment

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

Many lines mention the Avalon router.
Should we keep mention of a specific router name in the Dashboard, which is, AFAIK, not dedicated to a single router app/device ?

@mcmgj
Copy link

mcmgj commented Jun 16, 2021 via email

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.

3 participants