John Denker wrote: > On 11/27/2008 12:23 PM, Syd wrote: > > >> Since I've had to add a nasal inHG to Kpa conversion to most >> altimeters I've added , I decided maybe its something altimeter.cxx >> should do , so here is a small patch to do that. Could someone check and >> commit please ? >> > > This seems to be an unhappy medium. > > On the one hand, AFAICT this does not fully implement the > "setting-kpa" node. In particular, unpleasant things are > going to happen if somebody sets (via nasal) the setting-kpa > node. There is no code to maintain consistency if this happens. > It would be a burden on the documentation to outlaw this. > > On the other hand, I don't see how this is much of an > improvement over just reading the "setting-inhg" node and > applying the conversion factor each time. Floating-point > multiplication is cheap. > > Or am I missing something? > > Im not sure I understand what your getting at. This only converts setting-inhg to a setting-kpa property ... writing to it wont affect the setting-inhg , it will just be overwritten next loop.I thought manual settings should stay with setting-inhg to avoid problems down the line. Its just such a common display on the altimeters Ive done that I figured altimeter .cxx might as well do it.I think nasal is a great tool , but shouldnt have to do everything. If this is a problem I will continue to do it in Nasal , for each aircraft. Which is why I asked that it be checked first :) Cheers > ------------------------------------------------------------------------- > This SF.Net email is sponsored by the Moblin Your Move Developer's challenge > Build the coolest Linux based applications with Moblin SDK & win great prizes > Grand prize is a trip for two to an Open Source event anywhere in the world > http://moblin-contest.org/redirect.php?banner_id=100&url=/ > _______________________________________________ > Flightgear-devel mailing list > Flightgear-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/flightgear-devel > >
------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel