Re: [Flightgear-devel] FlightGear branch, next, updated. 59d400d58b082d583959eb8c27c3155eaa301888
Am 08.01.2012 16:11, schrieb Melchior FRANZ: > On Sunday 08 January 2012 05:09:07 Flightgear-commitlogs wrote: >> commit 246feef85fedd548f2c198831d7d6e7e3be53e12 >> Author: ThorstenB > >> FGNasalModelData isn't thread-safe. [...] It s*cks... ;-) > > It predates a separate model loader thread. If something sucks, > it's an imcomplete loader thread implementation (apart from > developers handing out grades without having much clue, that is). You shortened my commit message. The "s*cks" was not meant as a grade for FGNasalModelData or any of its authors. It was a final statement summarizing a larger issue as a whole. I had spent hours tracing sporadic memory corruption issues. These really s*ck, since they are not reliable to reproduce - and once you catch a corruption issue in the debugger, it's still difficult to find the origin. Of course, I have very little clue compared to you. You are right, as ever. However, despite being extremely little minded, I still knew that FGNasalModelData predates OSG. The part of my commit message, that you accidentally removed, also states that the issue was only introduced a short while ago, when the MP aircraft loading was moved to the separate loader thread. So, yes, things were fine before. And in the 'good old days' they were anyway. Melchior, yet again I deeply apologize if that commit has hurt your sensitive feelings. Sorry, sorry, sorry. It was not meant as a judgement of any of your involvement in FGNasalModelData, neither of your outstanding work on FlightGear, from the days when you still supported the project with useful contributions, instead of only digging through commit messages to find reasons to spill your poison. The legacy of your work is perfect and beyond any doubt. For the rest of the audience - here's the full commit message: > http://www.gitorious.org/fg/flightgear/commit/246feef85fedd548f2c198831d7d6e7e3be53e12 > commit 246feef85fedd548f2c198831d7d6e7e3be53e12 > Author: ThorstenB > Date: Sun Jan 8 13:28:49 2012 +0100 > > #553: MP-model-loading related segfault > MP aircraft are loaded by a separate OSG thread (introduced after > FG2.4.0). > The OSG thread also calls the "modelLoaded" callback. However, we mustn't > allow the OSG thread to call FGNasalModelData::modelLoaded directly: > FGNasalModelData isn't thread-safe. There are obvious issues > (_callCount++;), > tricky issues like calling the Nasal _parser_ or creating hashes and > modifying the global Nasal namespace. It doesn't use locks to protect > against another thread executing a Nasal context or running garbage > collection. It also executes Nasal code itself (the model's "load" hook), > which we also cannot allow in a separate thread... > This patch returns all Nasal parts of MP-aircraft loading (parsing, > module creation, execution) to the main thread, while keeping the > multi-threaded OSG part (loading of MP-aircraft model files itself). > The same issue exists with scenery models (see other commit). > > To summarize with 2 words: It s*cks... ;-) -- Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex infrastructure or vast IT resources to deliver seamless, secure access to virtual desktops. With this all-in-one solution, easily deploy virtual desktops for less than the cost of PCs and save 60% on VDI infrastructure costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox ___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
Re: [Flightgear-devel] FlightGear branch, next, updated. 59d400d58b082d583959eb8c27c3155eaa301888
On Sunday 08 January 2012 05:09:07 Flightgear-commitlogs wrote: > commit 246feef85fedd548f2c198831d7d6e7e3be53e12 > Author: ThorstenB > FGNasalModelData isn't thread-safe. [...] It s*cks... ;-) It predates a separate model loader thread. If something sucks, it's an imcomplete loader thread implementation (apart from developers handing out grades without having much clue, that is). m. :-P -- Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex infrastructure or vast IT resources to deliver seamless, secure access to virtual desktops. With this all-in-one solution, easily deploy virtual desktops for less than the cost of PCs and save 60% on VDI infrastructure costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox ___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
Re: [Flightgear-devel] PostGIS update @ Landcover-DB
Martin Spott wrote: > Related to the recent changes I sense that I've introduced an error in > "download.psp", will try to fix soon, Should be fixed - that's what you get from stupid duplication of code lines instead of cleanly putting identical stuff into a function Martin. -- Unix _IS_ user friendly - it's just selective about who its friends are ! -- -- Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex infrastructure or vast IT resources to deliver seamless, secure access to virtual desktops. With this all-in-one solution, easily deploy virtual desktops for less than the cost of PCs and save 60% on VDI infrastructure costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox ___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
Re: [Flightgear-devel] PostGIS update @ Landcover-DB
Gijs de Rooy wrote: > Ah! That's it. /me expected a nice dragable rectangle :-) No, not yet. Related to the recent changes I sense that I've introduced an error in "download.psp", will try to fix soon, Martin. -- Unix _IS_ user friendly - it's just selective about who its friends are ! -- -- Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex infrastructure or vast IT resources to deliver seamless, secure access to virtual desktops. With this all-in-one solution, easily deploy virtual desktops for less than the cost of PCs and save 60% on VDI infrastructure costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox ___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
Re: [Flightgear-devel] PostGIS update @ Landcover-DB
> click the lower left and upper right corner of a box and then the input > fields will get filled with the corresponding values. Ah! That's it. /me expected a nice dragable rectangle :-) -- Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex infrastructure or vast IT resources to deliver seamless, secure access to virtual desktops. With this all-in-one solution, easily deploy virtual desktops for less than the cost of PCs and save 60% on VDI infrastructure costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
Re: [Flightgear-devel] PostGIS update @ Landcover-DB
Gijs de Rooy wrote: > Not sure if I'm looking for something that you didn't add, but are the boxes > supposed to contain values? > http://mapserver.flightgear.org/map/?lon=4.75487&lat=52.31296&zoom=12&layers=0BTFFTTFF No, not by default. Enable the "Download-Box" layer, click the lower left and upper right corner of a box and then the input fields will get filled with the corresponding values. Note, the 144 square-degree limit still applies. Cheers, Martin. -- Unix _IS_ user friendly - it's just selective about who its friends are ! -- -- Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex infrastructure or vast IT resources to deliver seamless, secure access to virtual desktops. With this all-in-one solution, easily deploy virtual desktops for less than the cost of PCs and save 60% on VDI infrastructure costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox ___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
Re: [Flightgear-devel] PostGIS update @ Landcover-DB
Hi guys! Nice to see new features on the mapserver! > Martin wrote: > "download.psp" and "map/index.php" on the live page are now in sync > with GIT. Not sure if I'm looking for something that you didn't add, but are the boxes supposed to contain values? http://mapserver.flightgear.org/map/?lon=4.75487&lat=52.31296&zoom=12&layers=0BTFFTTFF Same behaviour on Firefox 9.0.1 and IE 9... (see screenshot: http://img546.imageshack.us/img546/7176/mapserver.png) Gijs -- Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex infrastructure or vast IT resources to deliver seamless, secure access to virtual desktops. With this all-in-one solution, easily deploy virtual desktops for less than the cost of PCs and save 60% on VDI infrastructure costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
Re: [Flightgear-devel] PostGIS update @ Landcover-DB
Adrian Musceac wrote: > I've placed a merge request with the changes. Now it works properly, at least > on my machine. Same here, thanks a lot for jumping in, I simply didn't get it - for whichever reason "download.psp" and "map/index.php" on the live page are now in sync with GIT. I've disabled the download box by default in order to prevent user confusion when they don't expect the clickable box feature. If you envision further improvement - I'm sure there's plenty of room - please go ahead, if you like. Cheers, Martin. -- Unix _IS_ user friendly - it's just selective about who its friends are ! -- -- Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex infrastructure or vast IT resources to deliver seamless, secure access to virtual desktops. With this all-in-one solution, easily deploy virtual desktops for less than the cost of PCs and save 60% on VDI infrastructure costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox ___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
Re: [Flightgear-devel] new Altimeter for Cessna c172p
Hello and thank you for your replies. Stuart Buchanan wrote: > I recall a while ago a recommendation from one of our graphics gurus > that it is more efficient to use textures rather than a number of > different objects without texture maps. This is counter-intuitive but explains why the AI colors are done this way. I will try this on the needles as well. It will reduce the number of objects, and simplify the .xml Question: shouldn't the AI knobs be illuminated? At present, they are not. Tuomas Kuosmanen wrote: > Just a random thought: does (could) it have both hPa and InHg scale in > the baro window? Would be very practical. Those exist in real life, > there sre just two baro scale windows on opposite sides of the gauge > face. Yes, I can try that. It means I'll have to limit the baro scale rotation to 180 degrees. Does anyone know what the scale min/max are on a real instrument? I have more questions: 1. Regarding optimization. is it OK to use N-gons for planar shapes, or should they be broken to quads and trigs? 2. Is it true that textures with an alpha channel are not efficient? In the case of a circular dial, is it better to cut the geometry and increase the polycount (1 quad -> 72 trigs) than use a texture map with a circular alpha? 3. Design priorities. My aim is to look at all the instruments to bring them to a consistent level. Then, add refinements like the 2 baro windows. If I compare the Cessna with the Beechcraft 1900, what strikes me is that the panel looks patchy. Would the team please tell me if you agree with this approach. 4. Git workflow. This is a question for Stuart and Tuomas. Once I commit a new instrument to the master branch in my clone repository, does git allow you to fetch that to your local master? So that you can inspect the change on your local machine? Thank you for your feedback. Stephan. -- Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex infrastructure or vast IT resources to deliver seamless, secure access to virtual desktops. With this all-in-one solution, easily deploy virtual desktops for less than the cost of PCs and save 60% on VDI infrastructure costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox ___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
Re: [Flightgear-devel] PostGIS update @ Landcover-DB
On Sunday, January 08, 2012 01:50:50 Martin Spott wrote: I've placed a merge request with the changes. Now it works properly, at least on my machine. Cheers, Adrian -- Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex infrastructure or vast IT resources to deliver seamless, secure access to virtual desktops. With this all-in-one solution, easily deploy virtual desktops for less than the cost of PCs and save 60% on VDI infrastructure costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox ___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
Re: [Flightgear-devel] PostGIS update @ Landcover-DB
On Sunday, January 08, 2012 01:50:50 Martin Spott wrote: > Martin Spott wrote: > > Cool ! I'll figure how to interface this with the download machinery. > > Ok, from my perspective the interface to "download.psp" works, please > check (fill in the values you prefer): > > > http://mapserver.flightgear.org/download.psp?xmin=6&xmax=7&ymin=50&ymax=51 > > but when I try to call the above download page from the > OpenLayers map: > > > http://mapserver.flightgear.org/testmap/?lon=-156.34932&lat=20.76679&zoom= > 11&layers=B0TFFFTFT > > then I still get the (hardcoded) default values, even though the > input fields in the OpenLayers map get filled properly. > > Any idea ? Heh, that's because I've ommited the name="xmin" part from the input fields :) So you can either add to each input field the name so that it is propagated through the request, or, if you're not really in a hurry, you can wait for me to also add Geoff's code that makes the selection click order irrelevant. Cheers, Adrian -- Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex infrastructure or vast IT resources to deliver seamless, secure access to virtual desktops. With this all-in-one solution, easily deploy virtual desktops for less than the cost of PCs and save 60% on VDI infrastructure costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox ___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
[Flightgear-devel] Re : flightgear.org DNS error
Hi all, De : Curtis Olson À : FlightGear developers discussions Envoyé le : Dimanche 8 Janvier 2012 1h38 Objet : Re: [Flightgear-devel] flightgear.org DNS error I'm not able to get through to easydns.com from here so maybe they are having issues? They've been rock solid for the 10-15 years I've been using them, so maybe they are having their turn with technical difficulties this evening? They have been "DDoSed" and are still under attack. Lost at least 2 of their DNS server: http://blog.easydns.org/ Oliver.-- Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex infrastructure or vast IT resources to deliver seamless, secure access to virtual desktops. With this all-in-one solution, easily deploy virtual desktops for less than the cost of PCs and save 60% on VDI infrastructure costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox___ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel