On 01/12/2007 05:41 PM, Georg Vollnhals wrote: > Hi John, > Generally spoken, I read all your posts with big interest.
:-) Thanks for the feedback. > don't worry - this was one of Melchior's usual jokes. We all saw he > corrected it in CVS thanks to your hint. OK, thanks for the explanation; I didn't understand the joke, and I wasn't expecting such a fast patch. And thanks for the fast patch! ==================== While we are patching model-howto.html, let me point out that it contains the following line: position-norm + offset -1 * factor 120 clamped to min/max -90:0 That needs repair in at least three ways: 1) The documentation says (and observation confirms) that the offset is applied *after* the factor. 2) It's offset-deg here, not offset. 3) The presentation violates time-honored algebraic conventions. It would be better typset as a table with two rows: position-norm * 120 + -1 clamped to [-90, 0] <property> * <factor> + <offset-deg> clamped to [<min>, <max>] =============== Tangential question: As Emerson put it, a foolish consistency is the hobgoblin of little minds, adored by little statesmen and philosophers and divines. But given that it was considered necessary to have <offset-deg> rather than <offset>, why do we have <min> rather than <min-deg>? Just curious. ================================================== On the more general subject of patching things: I was pleased by the high-speed response to the <offset-deg> issue. I was also surprised ... because there has been no discernible response to some other bugs I have reported, including ones where I went to a lot of trouble to find the bug and provide a patch. In particular, I would have thought that the fix to hsi.xml would have been completely noncontroversial, and I was sorta hoping it would have made it into cvs by now. Or, if there is something wrong with this patch, would somebody please explain and/or provide something better? http://www.av8n.com/fly/fgfs/hsi.xml.htm http://www.av8n.com/fly/fgfs/hsi.xml.diff Similarly, the handling of non-unique navaid identifiers was on the TODO list. Now it's done ... or at least it's done chez moi. If other folks are to benefit from this work, somebody will have to propagate the changes into the cvs tree. http://www.av8n.com/fly/fgfs/nearest-fix.diff Ditto for the new & improved location-in-air popup. I know some people are using it ... but evidently only people who live /beyond/ the bleeding edge. If this work is to benefit people who live only /at/ the bleeding edge, let alone the hoi polloi, somebody will have to propagate the new thing into the cvs tree at some point. http://www.av8n.com/fly/fgfs/location-in-air.xml.htm > It is great > to have someone with real-life knowledge working together with the > developers. :-) ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel