On Fri, 22 Jun 2007 23:27:51 +0100 "Vivian Meazza" <[EMAIL PROTECTED]> wrote:
> Syd > > > Sent: 22 June 2007 20:18 > > To: FlightGear developers discussions > > Subject: Re: [Flightgear-devel] [ANN] OSG - Improved Weather Radar > > > > > > On Fri, 22 Jun 2007 16:56:34 +0100 > > "Vivian Meazza" <[EMAIL PROTECTED]> wrote: > > > > > > > > > > > > -----Original Message----- > > > Syd > > > > > > > Sent: 22 June 2007 15:43 > > > > To: FlightGear developers discussions > > > > Subject: Re: [Flightgear-devel] [ANN] OSG - Improved Weather Radar > > > > > > > > > > > > On Fri, 22 Jun 2007 13:42:16 +0200 > > > > Melchior FRANZ <[EMAIL PROTECTED]> wrote: > > > > > > > > > * Vivian Meazza -- Wednesday 13 June 2007: > > > > > > Tim Moore has been hard at work recently (with the smallest of > > > > > > inputs by me), and has ported the improved weather > > radar already > > > > > > available for plib to OSG. > > > > > > > > > > No objections and other comments since the patches were > > > > published on > > > > > 2007/06/20. Because of the nearing release (not that we have the > > > > > slightest idea when this could be :-) and the nature of > > the patch I > > > > > want to give developers one last chance to object: if > > > > nobody does that > > > > > until tomorrow 2007/06/23 20:00 GMT, then I will: > > > > > > > > > > (a) apply those radar patches to sg and fg for osg and plib > > > > > (b) comment out the "delete rt" in > > > > src/Instrumentation/od_gauge.cx:89 > > > > > > > > > > > > > > > > > > > > PRO ------------------------------------------------------------ > > > > > > > > > > + we have a nice c++ radar implementation in both > > branches, which > > > > > handles arbitrary numbers of AI/MP aircraft, ships, TACAN > > > > > emitting and other objects > > > > > > > > > > + we can drop the quite clumsy and limited & limiting XML radar > > > > > implementation > > > > > > > > > > + fixes a bug in AIManager (ask Vivian :-) > > > > > > > > > > + instantiates impact sub-submodels in correct order > > > > > > > > > > > > > > > > > > > > CONTRA --------------------------------------------------------- > > > > > > > > > > - bigger commit despite the near(?) release, with potential risk > > > > > to break something > > > > > > > > > > BUT: + the patch touches only files that can hardly have side > > > > > effects on other subsystems, and isn't executed at all > > > > > when aircraft without od_gauge/radar are used (which > > > > > is the vast majority). So even if there'd be problems, > > > > > they would only affect the E3B, the T38(?), and ... the > > > > > harrier? And even then one could comment out the radar > > > > > instrument in the XML file and avoid all problems. > > > > > > > > > > + the patches were tested by, at least, Vivian, AJ, > > > > > Csaba "Jester"(?) and me, and found functional and not > > > > > causing problems, except the following (AJ and I): > > > > > > > > > > - requires to comment out the destruction of the RTT class to > > > > > avoid crashes on exit on (some?) nVidia cards. That's hackish, > > > > > > > > > > BUT: + that's exactly what the 3D clouds are doing > > since years! > > > > > They don't destruct the RTT class either! Nobody has > > > > > reported problems that could be linked to that, none > > > > > of the developers has observed such problems (AFAIK). > > > > > > > > > > + that's exactly what the TestRenderTexture.cxx test > > > > > application by the very author of the RenderTexture > > > > > class does. He doesn't destruct the class > > either (except > > > > > before creating a new one during mode changes on user > > > > > request). > > > > > > > > > > + it can be assumed that the card frees this resource > > > > > like all others, when the context is destroyed, so the > > > > > buggy freeing operation via glXDestroyPbuffer() should > > > > > be optional in this case. > > > > > > > > > > m. > > > > > > > > Hi , I vote for adding it , and I've had that shutdown error > > > > since I first used the wxradar ,it's not a new one here... > > > > Just my 2 cents worth :). Cheers > > > > > > > > > > That's very interesting information. We suspected that this was a > > > longstanding problem, but had no evidence. And does Melchior's fix, > > > fix it for you? > > > > > > V. > > > > > Hi Vivian, > > No I haven't tried the updated version yet ... anxiously > > awaiting CVS version ;).But I can try it first if you like , > > and see what I get ... Cheers > > -- > > > > It would be helpful if you could comment out "delete rt" in > src/Instrumentation/od_gauge.cx around line #89, and tell us if that fixes > the problem in the old code. > > Thanks > > Vivian > Hi again, I checked the source code , recent update , and "delete rt" is already commented out , so I recompiled and tried my Aerostar 700, which I recently added a weather radar to , and no more error message when I exit FG , so that must have done the trick ... Cheers -- syd & sandy <[EMAIL PROTECTED]> ------------------------------------------------------------------------- This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now. http://sourceforge.net/powerbar/db2/ _______________________________________________ Flightgear-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/flightgear-devel
