Cruisers Forum
 


Join CruisersForum Today

Reply
 
Thread Tools Rate Thread Display Modes
Old 22-06-2014, 08:01   #166
Registered User

Join Date: Jul 2010
Location: Monastir, Tunisia
Boat: Amel Sharki
Posts: 1,777
Re: OpenCPN Beta Version 3.3.1731 Released

Quote:
Originally Posted by bdbcat View Post
Gerhard....

Latest GRIB is broken in what way?

Thanks
Dave
Does not show any overlays nor any data.
Reverted back and all is working again.

Gerhard
__________________

__________________
CarCode is online now   Reply With Quote
Old 22-06-2014, 08:07   #167
Registered User

Join Date: Jul 2010
Location: Monastir, Tunisia
Boat: Amel Sharki
Posts: 1,777
Re: OpenCPN Beta Version 3.3.1731 Released

BTW: GshhsPolyReader::crossing1() crashes also.

I'm really not amused about Sean.

Gerhard
__________________

__________________
CarCode is online now   Reply With Quote
Old 22-06-2014, 10:23   #168
Marine Service Provider
 
bdbcat's Avatar

Join Date: Mar 2008
Posts: 4,884
Re: OpenCPN Beta Version 3.3.1731 Released

Gerhard...

1. Grib: Current github works for me on Windows and Linux. See pix. Not tested Mac yet. That's next.

2. Acknowledge the crash on gshhs.cpp
If you want to test further, you may patch gshhs.cpp and remove all occurrences of "assert(...), or replace them with "wxASSERT(...)

Dave
Attached Thumbnails
Click image for larger version

Name:	grib1.jpg
Views:	58
Size:	123.3 KB
ID:	83550  
__________________
bdbcat is offline   Reply With Quote
Old 22-06-2014, 10:58   #169
Registered User

Join Date: Jul 2010
Location: Monastir, Tunisia
Boat: Amel Sharki
Posts: 1,777
Re: OpenCPN Beta Version 3.3.1731 Released

Thanks, assert() has been already patched here.
Dunno why grib 2.3 doesn't work. Got back to 2.2 and it is OK.

Gerhard
__________________
CarCode is online now   Reply With Quote
Old 22-06-2014, 11:55   #170
Registered User

Join Date: Sep 2009
Location: Angers - France
Boat: Beneteau First 29 Ptizef
Posts: 743
Re: OpenCPN Beta Version 3.3.1731 Released

New git hub today (windows )
- OpenCPN do not compile (problem symbol extern on GshhsPolyReader::crossing1Init(void) )
- Grib-Pi compile OK but do not work with previous git's opencpn.exe
- nothing displayed (barbed arrows, colored overlays etc ..
- crash if I try to change any setting

Jean Pierre
__________________
Ptizef is offline   Reply With Quote
Old 22-06-2014, 12:44   #171
Registered User

Join Date: Jul 2010
Location: Monastir, Tunisia
Boat: Amel Sharki
Posts: 1,777
Re: OpenCPN Beta Version 3.3.1731 Released

Jean Pierre,

la meme choose ici avec Mac OS X.
Je ne suis pas content avec les modificationes d'un certaine personne.

Gerhard
__________________
CarCode is online now   Reply With Quote
Old 22-06-2014, 13:05   #172
Registered User

Join Date: Jul 2010
Location: Monastir, Tunisia
Boat: Amel Sharki
Posts: 1,777
Re: OpenCPN Beta Version 3.3.1731 Released

BTW Since implementing "Merge pull request #135 from seandepagnier/master" my CPU usage climbed up to 38 percent.
I got it down to 3-4 percent before.

I vote to: please do not kill us anymore with these "optimizations".
There are still some other tasks like a final s63 plugin.

Gerhard
__________________
CarCode is online now   Reply With Quote
Old 22-06-2014, 13:52   #173
bcn
Registered User

Join Date: May 2011
Location: underway whenever possible
Boat: Rangeboat 39
Posts: 2,766
Re: OpenCPN Beta Version 3.3.1731 Released

Gerhard,

don't understand your "panic mode".

Dave laid out the next steps in the post #160 in this thread -
and regressions in a case like this are no surprise.
Errors neither - that's what testing of a beta is about, isn't it?

So your testing helps, yes.

Hubert
__________________
bcn is offline   Reply With Quote
Old 22-06-2014, 14:02   #174
Registered User

Join Date: Jul 2010
Location: Monastir, Tunisia
Boat: Amel Sharki
Posts: 1,777
Re: OpenCPN Beta Version 3.3.1731 Released

Hubert,

es ist keine Panik, aber schlichtweg eine Unverschämheit wie dieser Kerl die Sourcen kaputt macht. Vielleicht verstehst Du das nicht so...

Gerhard
__________________
CarCode is online now   Reply With Quote
Old 22-06-2014, 14:12   #175
bcn
Registered User

Join Date: May 2011
Location: underway whenever possible
Boat: Rangeboat 39
Posts: 2,766
Re: OpenCPN Beta Version 3.3.1731 Released

Quote:
Originally Posted by CarCode View Post
Hubert,

es ist keine Panik, aber schlichtweg eine Unverschämheit wie dieser Kerl die Sourcen kaputt macht. Vielleicht verstehst Du das nicht so...

Gerhard
translation:

"this is no panic but it is just impertinent how this guy is destroying the sources.
Perhaps you don't understand this in this way.."


--> #160 of this thread...

btw.: S63 works with either of the beta versions and people are sailing with it.
And the nice thing about Open Software is that we don't have deadlines.
So why being impatient? Or nervous?

Hubert
__________________
bcn is offline   Reply With Quote
Old 22-06-2014, 14:25   #176
Registered User

Join Date: Jul 2010
Location: Monastir, Tunisia
Boat: Amel Sharki
Posts: 1,777
Re: OpenCPN Beta Version 3.3.1731 Released

Hubert,

I think you don't get it.
S63 plugin is still beta and users don't use betas especially if they have to pay for it.
Instead of eliminating bugs there are always new bugs in the main code.
That is very frustating.

Gerhard
__________________
CarCode is online now   Reply With Quote
Old 22-06-2014, 14:41   #177
bcn
Registered User

Join Date: May 2011
Location: underway whenever possible
Boat: Rangeboat 39
Posts: 2,766
Re: OpenCPN Beta Version 3.3.1731 Released

Well,

the idea is that the S63 PI will go 1.0 with the next beta.
And on the other hand if somebody now wants to use S63 charts - that works.
Makes no difference for us if there are 15, 150 or 15.000 users of the PI.
But we are implied to have it stable and reliable, be assured.

And if the number of bugs increases or is getting more comfortable thats just the normal process. Remember the Debian release cycles just in case.

No reason for frustration for me - just testing and reporting in order to add my grain of salt to the development cycle.

Good night

Hubert
__________________
bcn is offline   Reply With Quote
Old 22-06-2014, 21:11   #178
Registered User
 
boat_alexandra's Avatar

Join Date: Aug 2009
Location: chesapeake bay
Boat: bristol 27
Posts: 2,816
Re: OpenCPN Beta Version 3.3.1731 Released

Quote:
Originally Posted by bdbcat View Post
Gerhard...

1. Grib: Current github works for me on Windows and Linux. See pix. Not tested Mac yet. That's next.

2. Acknowledge the crash on gshhs.cpp
If you want to test further, you may patch gshhs.cpp and remove all occurrences of "assert(...), or replace them with "wxASSERT(...)

Dave
Dave, does using wxASSERT instead of assert fix the crashes? assert was already being used in gshhs.cpp so I assumed it is ok to use there.

Quote:
Originally Posted by Ptizef View Post
Hi
I tried to find the commit using git-bisect

I found this is between
this commit :
12/05/2014
3.3.1712
100fdbccce6865ebb356d67bad289ff5ff8f6c73
which is the last I can compile and test with OK result

And this one :

15/05/2014
Correct glCanvas setup for Windows proc addr location search.
2e0754af9bf41347cf4cb6fd6b108c8880b3011d

with is the fisrt I can compile and test NO OK


Between them, I was unable to test because either they do not compile or they crash


But the more important is I think I've found a fix for the problem with this patch :


//

@@ -1430,7 +1432,7 @@ void GRIBOverlayFactory:rawGLTexture( GLuint texture, int width, int height,
glEnable(GL_TEXTURE_RECTANGLE_ARB);
glBindTexture(GL_TEXTURE_RECTANGLE_ARB, texture);

- glDisable( GL_MULTISAMPLE );

glEnable(GL_BLEND);
glBlendFunc(GL_SRC_ALPHA, GL_ONE_MINUS_SRC_ALPHA);



//
After suppressing this line which seemed curious, I was unable to reproduce the display problem on both my machines


I will post a pull request for that patch


Jean Pierre
The glDisable should not hurt anything, but also shouldn't be there, so you are right to delete that line. Perhaps your driver uses multisampling to emulate something.

Quote:
Originally Posted by Ptizef View Post
New git hub today (windows )
- OpenCPN do not compile (problem symbol extern on GshhsPolyReader::crossing1Init(void) )
windows handles function linkages in a different way, so the build error only shows up there. I have fixed the problem, sorry for the trouble. I have pushed the change to my fork (github.com/seandepagner/OpenCPN) and will post a pull request.
Quote:
- Grib-Pi compile OK but do not work with previous git's opencpn.exe
- nothing displayed (barbed arrows, colored overlays etc ..
- crash if I try to change any setting

Jean Pierre
It works for me, the changes are to use polar interpolations instead of vector interpolation, and provide time slices for moving gribs.

My guess is that the new routine breaks only on certain grib files. I will try with more grib files, but if you could post the one that isn't working it would help.
__________________
boat_alexandra is offline   Reply With Quote
Old 22-06-2014, 22:17   #179
Marine Service Provider
 
bdbcat's Avatar

Join Date: Mar 2008
Posts: 4,884
Re: OpenCPN Beta Version 3.3.1731 Released

Sean...

1. wxASSERT fixes the assert crash. Dunno why, exactly.

2. I have fixed the Windows linkage problems in gshhs, and will be in my next batch of commits. So no need for a pull request. Actually, the problem was a bad diff in the git commit. The entire method GshhsPolyReader::crossing1Init() went missing. I have no idea how linux actually linked the full binary....

3. I also would like to see specific GRIB files that cause trouble, so I can add them to my test pool.

4. I am close to a Beta release, tomorrow probably. So you may wish to hold off on any tweaks for a couple of days, please.

Thanks
Dave
__________________
bdbcat is offline   Reply With Quote
Old 23-06-2014, 00:43   #180
Registered User

Join Date: Sep 2009
Location: Angers - France
Boat: Beneteau First 29 Ptizef
Posts: 743
Re: OpenCPN Beta Version 3.3.1731 Released

Hi Dave, Sean

Ok I think I have understood
It work ( at least the first test ) with Saildocs's files
But it do not work with Zygrib's files
I faced that problem in the past and found they have a difference. I think they have a negative DJ or something like that don't remember exactly and no time this morning to search. I joint a file as example

I confirm : No display and crash when changing setting and interpolate is selected

Jean Pierre
__________________

__________________
Ptizef is offline   Reply With Quote
Reply

Tags
enc, lease, opencpn

Thread Tools
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
OpenCPN Beta Version 3.1.814 Released bdbcat OpenCPN 185 14-09-2012 08:43
OpenCPN Beta Version 3.1.802 Released bdbcat OpenCPN 158 14-08-2012 11:07
OpenCPN Beta Version 3.1.714 Released bdbcat OpenCPN 91 01-08-2012 18:08



Copyright 2002- Social Knowledge, LLC All Rights Reserved.

All times are GMT -7. The time now is 06:04.


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

ShowCase vBulletin Plugins by Drive Thru Online, Inc.