Cruisers Forum
 


Join CruisersForum Today

Reply
 
Thread Tools Search this Thread Rate Thread Display Modes
Old 14-03-2018, 09:23   #106
Registered User
 
rgleason's Avatar

Join Date: Mar 2012
Location: Boston, MA
Boat: 1981 Bristol 32 Sloop
Posts: 10,910
Re: New Plugin NmeaConvertor

Updated to "fix compiler warnings"
https://github.com/rgleason/NmeaConverter_pi/releases
__________________

rgleason is offline   Reply With Quote
Old 14-03-2018, 15:56   #107
Registered User

Join Date: Mar 2016
Location: Petaluma, CA
Boat: Morgan 382
Posts: 69
Re: New Plugin NmeaConvertor

I am playing with this on 2 machines. Both are running 4.8.2, installed from the .exe on the download page.

On one machine I am running NMEA converter which I compiled myself. (compiled on a separate 3rd machine then I copied the dll over) It isn't the latest update which accepts wildcards, but about 2 weeks old. It seems to work fine, although I have not run it for extended periods.

On the other machine I installed NMEA converter from the .exe that was posted by Rick. It freezes opencpn when I click on "new" from the preferences dialog.

Let me know how I can collect information to help track the issue. One difference I now realize is that on the machine that it freezes on, I don't have any incoming NMEA data presently.

Somewhat related, I pursued this for TWS and TWA, and the long and complex formula posted here seems to work fine for that on the machine this plugin is working on.
__________________

wholybee is offline   Reply With Quote
Old 14-03-2018, 18:12   #108
Registered User
 
rgleason's Avatar

Join Date: Mar 2012
Location: Boston, MA
Boat: 1981 Bristol 32 Sloop
Posts: 10,910
Re: New Plugin NmeaConvertor

Wollybee, I don't get a crash when I hit "New" but I do get a crash when I try to run a formula while playing an data stream.

I have tried to compile "out of tree" or "standalone mode" and I'm unable to create a debug dll, possibly because the CMAKE is not fully developed?

I then tried to move the nmeaconverter_pi "into the tree" and then run debug to get the required debug dll. That does not seem to work either, which makes me think it is a CMAKE problem.

So in short I cannot debug it, but I can confirm it is not working.
rgleason is offline   Reply With Quote
Old 15-03-2018, 05:16   #109
Registered User
 
rgleason's Avatar

Join Date: Mar 2012
Location: Boston, MA
Boat: 1981 Bristol 32 Sloop
Posts: 10,910
Re: New Plugin NmeaConvertor

The last update to packaging & Cmake was June6 &7, 2015.
https://github.com/RooieDirk/NmeaCon...commits/master

I am going to try a fresh git clone to make sure there isn't some anomaly, but github says my repos is identical to Dirks.
rgleason is offline   Reply With Quote
Old 15-03-2018, 06:33   #110
Registered User
 
rgleason's Avatar

Join Date: Mar 2012
Location: Boston, MA
Boat: 1981 Bristol 32 Sloop
Posts: 10,910
Re: New Plugin NmeaConvertor

Dirk
What Plugin API version number does this use?
I have been using Plugin API
pi113 (1.13) = OpenCPN 4.2 ←SVG Icons added (gtk,expat,vc,etc)
And it compiles with no complaint.
Has it changed?

I only found in a search for "API"

#include "version.h"

#define MY_API_VERSION_MAJOR 1
#define MY_API_VERSION_MINOR 8

#include "ocpn_plugin.h"

I certainly do not understand this number? 1.8? We don't have such a Plugin API that I know of.
rgleason is offline   Reply With Quote
Old 15-03-2018, 06:42   #111
Registered User
 
rgleason's Avatar

Join Date: Mar 2012
Location: Boston, MA
Boat: 1981 Bristol 32 Sloop
Posts: 10,910
Re: New Plugin NmeaConvertor

Using Plugin API Version 1.13 it compiles, but that does not make it right.

I get 33 warnings which might help (since I can't make this run in debug) and a comparison of the various CMAKE files with WMM_pi shows that some If WIN statements are missing for standalone and in tree compiling.



Quote:
Build succeeded.

"C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\ALL_BUILD.
vcxproj" (default target) (1) ->
"C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconver
ter_pi.vcxproj" (default target) (3) ->
(ClCompile target) ->
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(26): warning C4273: 'wxEcEngine::wxEcEngine' : inconsistent dll linkage [C:\
Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconverter_
pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(37): warning C4273: 'wxEcEngine::~wxEcEngine' : inconsistent dll linkage [C:
\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconverter
_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(42): warning C4273: 'wxEcEngine::GetFormula' : inconsistent dll linkage [C:\
Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconverter_
pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(47): warning C4273: 'wxEcEngine::GetIndicator' : inconsistent dll linkage [C
:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconverte
r_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(55): warning C4273: 'wxEcEngine::GetLastError' : inconsistent dll linkage [C
:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconverte
r_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(60): warning C4273: 'wxEcEngine::GetLastResult' : inconsistent dll linkage [
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconvert
er_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(85): warning C4273: 'wxEcEngine::Reset' : inconsistent dll linkage [C:\Users
\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconverter_pi.vc
xproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(106): warning C4273: 'wxEcEngine::TranslateError' : inconsistent dll linkage
[C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconve
rter_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(126): warning C4273: 'wxEcEngine:eleteConstant' : inconsistent dll linkage
[C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconve
rter_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(131): warning C4273: 'wxEcEngine::GetConstant' : inconsistent dll linkage [C
:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconverte
r_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(158): warning C4273: 'wxEcEngine::GetConstantCount' : inconsistent dll linka
ge [C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeacon
verter_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(163): warning C4273: 'wxEcEngine::ListAllConstants' : inconsistent dll linka
ge [C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeacon
verter_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(175): warning C4273: 'wxEcEngine::ResetConstants' : inconsistent dll linkage
[C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconve
rter_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(186): warning C4273: 'wxEcEngine::SetConstant' : inconsistent dll linkage [C
:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconverte
r_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(230): warning C4273: 'wxEcEngine::BracketLookup' : inconsistent dll linkage
[C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconver
ter_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(268): warning C4273: 'wxEcEngine::ConvertToRadian' : inconsistent dll linkag
e [C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconv
erter_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(279): warning C4273: 'wxEcEngine::CountChar' : inconsistent dll linkage [C:\
Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconverter_
pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(288): warning C4273: 'wxEcEngine::FindOperator' : inconsistent dll linkage [
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconvert
er_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(297): warning C4273: 'wxEcEngine::Function2ID' : inconsistent dll linkage [C
:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconverte
r_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(311): warning C4273: 'wxEcEngine::FunctionLookUp' : inconsistent dll linkage
[C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconve
rter_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(337): warning C4273: 'wxEcEngine::IsOperator' : inconsistent dll linkage [C:
\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconverter
_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(344): warning C4273: 'wxEcEngine::IsParitySign' : inconsistent dll linkage [
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconvert
er_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(357): warning C4273: 'wxEcEngine::IsValid' : inconsistent dll linkage [C:\Us
ers\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconverter_pi
.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(390): warning C4273: 'wxEcEngine::LeftPool' : inconsistent dll linkage [C:\U
sers\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconverter_p
i.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(397): warning C4273: 'wxEcEngine::NextOperator' : inconsistent dll linkage [
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconvert
er_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(406): warning C4273: 'wxEcEngine::Simplify' : inconsistent dll linkage [C:\U
sers\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconverter_p
i.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(464): warning C4273: 'wxEcEngine::ApplyFunction' : inconsistent dll linkage
[C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconver
ter_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(624): warning C4273: 'wxEcEngine::Compute' : inconsistent dll linkage [C:\Us
ers\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconverter_pi
.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(635): warning C4273: 'wxEcEngine::evalexp' : inconsistent dll linkage [C:\Us
ers\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconverter_pi
.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(670): warning C4273: 'wxEcEngine::evalf' : inconsistent dll linkage [C:\User
s\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconverter_pi.v
cxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(764): warning C4273: 'wxEcEngine:erivate' : inconsistent dll linkage [C:\U
sers\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconverter_p
i.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(788): warning C4273: 'wxEcEngine::SetFormula' : inconsistent dll linkage [C:
\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconverter
_pi.vcxproj]
C:\Users\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\src\ec_engine.c
pp(805): warning C4273: 'wxEcEngine::StackIt' : inconsistent dll linkage [C:\Us
ers\Frederick\Documents\GitHub\o-plugin\NmeaConverter_pi\build\nmeaconverter_pi
.vcxproj]

33 Warning(s)
0 Error(s)
rgleason is offline   Reply With Quote
Old 15-03-2018, 06:56   #112
Registered User
 
rgleason's Avatar

Join Date: Mar 2012
Location: Boston, MA
Boat: 1981 Bristol 32 Sloop
Posts: 10,910
Re: New Plugin NmeaConvertor

With fresh git clone, compile, opencpn.lib version PI-API 1.13, install, had to rename opencpn.ini because previous version of nmea had made opencpn so it would not load properly (Toolbar Icons missing). so then started opencpn and a new opencpn.ini was created. I could then Enable the plugin.

Then when I clicked "New" eventually after long blue circle spinning, I got an OpenCPN failure and Windows notice.
rgleason is offline   Reply With Quote
Old 15-03-2018, 07:02   #113
Registered User
 
rgleason's Avatar

Join Date: Mar 2012
Location: Boston, MA
Boat: 1981 Bristol 32 Sloop
Posts: 10,910
Re: New Plugin NmeaConvertor

Tried compile with v1.15 of opencpn.lib and installed. It fails in the same way, when I hit "new"
This is as far as I can take it, as it does not compile in debug mode either.
rgleason is offline   Reply With Quote
Old 20-03-2018, 18:56   #114
Registered User
 
rgleason's Avatar

Join Date: Mar 2012
Location: Boston, MA
Boat: 1981 Bristol 32 Sloop
Posts: 10,910
Re: New Plugin NmeaConvertor

The most recent version of this plugin, when compiled for Windows and run, hangs up and fails Opencpn when you hit "New" to enter a nmea conversion formula.

I have reported this earlier.. The debug version does not work either.
It would be great if we could get this fixed.

Dirk has just added a wildcard option.

Github https://github.com/rgleason/NmeaConverter_pi

Documentation
https://opencpn.org/wiki/dokuwiki/do...nmea_converter

I'd really appreciate a programmer looking at this. Is this a problem in Linux too?
rgleason is offline   Reply With Quote
Old 21-03-2018, 11:41   #115
Registered User
 
rgleason's Avatar

Join Date: Mar 2012
Location: Boston, MA
Boat: 1981 Bristol 32 Sloop
Posts: 10,910
Re: New Plugin NmeaConvertor

Mike R. [Rasbats] has figured out that RooieDirk's version is not the one that compiled for windows... The one compiled for windows that works is Pavels [nohal] "Bump version" commit.

I believe Mike will be comparing the two versions to come up with a fix. Thanks again Mike!
rgleason is offline   Reply With Quote
Old 21-03-2018, 12:26   #116
Registered User

Join Date: May 2012
Posts: 474
Re: New Plugin NmeaConvertor

Rick ...

I did not figure anything out!

If an issue is being discussed on GitHub please can we keep the conversation there.

Thanks.
Mike
Rasbats is offline   Reply With Quote
Old 21-03-2018, 12:49   #117
Registered User
 
rgleason's Avatar

Join Date: Mar 2012
Location: Boston, MA
Boat: 1981 Bristol 32 Sloop
Posts: 10,910
Re: New Plugin NmeaConvertor

Ok, I'll keep it there, but we now know that the one that works for windows is pavel's "bump version" commit, so that's much better than what we knew before, I was banging my head on that one.
rgleason is offline   Reply With Quote
Old 21-03-2018, 15:30   #118
Registered User
 
rgleason's Avatar

Join Date: Mar 2012
Location: Boston, MA
Boat: 1981 Bristol 32 Sloop
Posts: 10,910
Re: New Plugin NmeaConvertor

We've had to step back one commit, as the last commit to fix compiler warnings does not work. So this version is the previous version which "adds wildcards" which will be very useful if you don't know a devices id. So the note in the manual about this

Add Wildcards - NmeaConverter Manual

Beta download
nmeaconverter_pi-0.2-add-wildcard-win32

We really need this tested carefully by some pros. Please let us know how it is working, or we will not know and Dirk says he will not be able to release it from Beta.

Wollybee I hope you'll give this one a try.
rgleason is offline   Reply With Quote
Old 22-03-2018, 13:51   #119
Registered User
 
rgleason's Avatar

Join Date: Mar 2012
Location: Boston, MA
Boat: 1981 Bristol 32 Sloop
Posts: 10,910
Re: New Plugin NmeaConvertor

Some suggestions about how to test nmeaconverter if you are not on your boat and need nmea data. Additional examples and formulas are welcome, we will add them to the manual!
rgleason is offline   Reply With Quote
Old 23-03-2018, 22:55   #120
Registered User

Join Date: Mar 2016
Location: Petaluma, CA
Boat: Morgan 382
Posts: 69
Re: New Plugin NmeaConvertor

Quote:
Originally Posted by rgleason View Post
We've had to step back one commit, as the last commit to fix compiler warnings does not work. So this version is the previous version which "adds wildcards" which will be very useful if you don't know a devices id. So the note in the manual about this

Add Wildcards - NmeaConverter Manual

Beta download
nmeaconverter_pi-0.2-add-wildcard-win32

We really need this tested carefully by some pros. Please let us know how it is working, or we will not know and Dirk says he will not be able to release it from Beta.

Wollybee I hope you'll give this one a try.
Wildcards don't seem to work for me. When I first enter a ?, the output line changes to "nodata" and then a few seconds later OpenCPN crashes.
-Warren
__________________

__________________
-Warren
wholybee is offline   Reply With Quote
Reply

Tags
nmea, plug

Thread Tools Search this Thread
Search this Thread:

Advanced Search
Display Modes Rate This Thread
Rate This Thread:

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off
Trackbacks are Off
Pingbacks are Off
Refbacks are Off


Similar Threads
Thread Thread Starter Forum Replies Last Post
Calculator_pi New Finished Plugin for OpenCPN SaltyPaws OpenCPN 122 08-03-2018 09:34
New IAC Fleetcode Plugin for OpenCPN CBorchardt OpenCPN 73 02-02-2018 07:35
New Plugin Development on Linux globalkeith OpenCPN 5 03-05-2012 07:16
New History Plugin for OpenCPN boat_alexandra OpenCPN 11 04-10-2010 14:11



Copyright 2002- Social Knowledge, LLC All Rights Reserved.

All times are GMT -7. The time now is 22:02.


Google+
Powered by vBulletin® Version 3.8.8 Beta 1
Copyright ©2000 - 2018, vBulletin Solutions, Inc.
Social Knowledge Networks
Powered by vBulletin® Version 3.8.8 Beta 1
Copyright ©2000 - 2018, vBulletin Solutions, Inc.

ShowCase vBulletin Plugins by Drive Thru Online, Inc.