On Fri, Mar 25, 2011 at 7:44 AM,  Thorsten Renk wrote:
>> The "local weather" menu option is first before the other local weather
>> options.  This does open up a dialog box where you do have to create
>> clouds
>> 1-by-1.  Looking at this, it appears that the intent of this is more for
>> debugging.  Maybe it would make sense to move this option over to the
>> debugging menu if that is the case?
>
> That's actually a good point and I was about to ask a similar thing for
> the release version.
>
> The main use of the 'Local Weather' menu item is development. I use it
> when I introduce a new texture to test which cloud spacing, which degree
> of randomness and such things a layer composed of these clouds needs to
> look natural - once I have the parameters, I code a Nasal call to the same
> effect. It's also useful for benchmark tests because it allows to create a
> well-defined configuration of clouds. Finally, at some point Patrice Poly
> used it when he did some experiments with hires textures as an easy way to
> test how the extracted textures look in the scenery.
>
> It doesn't really have user applications or debug value. So my original
> suggestion would have been to rename 'Local Weather Tiles' to 'Local
> Weather' and to comment out the current 'Local Weather' in the
> menubar.xml. The small number of people who may actually need it for
> development can then simply edit menubar.xml and have it again, the rest
> doesn't need to see it, because it is confusing.
>
> However, if someone sees value in having it in the debug section or
> somewhere else, then I suggest to move it.

I prefer the idea of renaming and moving, rather than commenting out
in the menubar.xml file.

I think there is value in retaining the dialog for those interested
in creating new cloud layers in the future, so having it in the Debug menu
makes sense (though perhaps "Debug" should be renamed "Development").

The Debug menu is not used by most users, and is not documented in the
manual, so I don't see any problem in adding it there, even if it won't be used
regularly.

-Stuart

------------------------------------------------------------------------------
Enable your software for Intel(R) Active Management Technology to meet the
growing manageability and security demands of your customers. Businesses
are taking advantage of Intel(R) vPro (TM) technology - will your software 
be a part of the solution? Download the Intel(R) Manageability Checker 
today! http://p.sf.net/sfu/intel-dev2devmar
_______________________________________________
Flightgear-devel mailing list
Flightgear-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/flightgear-devel

Reply via email to