API Change Proposal - Re: MSAA and Scene anti aliasing

[Richard Bair]

Actually that is exactly what I was hoping DEFAULT would mean -- some kind of typical option for the particular platform (maybe even disabled on some!).

Richard

On Jul 23, 2013, at 1:41 PM, Felix Bembrick <felix.bembrick at gmail.com> wrote:

> I am not sure I like the name of the value DEFAULT.  To me "default"
> implies some kind of "typical" option for the particular platform which
> could conceivably be very different on each platform.  For example, on my
> Windows super-computer with ultra fast GPU the "default" would logically be
> something like 16x whereas on my iPhone I would expect the default to be
> using only 2x.  The comment for DEFAULT implies that it's really more of a
> BALANCED or MODERATE setting that would have the same characteristics on
> each platform.
> 
> 
> On 24 July 2013 06:29, Chien Yang <chien.yang at oracle.com> wrote:
> 
>> Hi all,
>> 
>>    We appreciate all the feedback you have contributed to this topic.
>> After listening to the feedback and an internal discussion, we would like
>> to propose a minor change to the API for supporting scene anti-aliasing. We
>> intentionally choose not to expose the number of samples and techniques
>> used in this release, but this doesn't preclude future addition when the
>> time is right for more options. This change will be tracked by RT-31878 (
>> https://javafx-jira.kenai.**com/browse/RT-31878<https://javafx-jira.kenai.com/browse/RT-31878>
>> ):
>> 
>> Anti-aliasing API Change Proposal:
>> 
>> Constructors remove:
>> public Scene(Parent root, double width, double height, boolean
>> depthBuffer, boolean antiAliasing)
>> public SubScene(Parent root, double width, double height, boolean
>> depthBuffer, boolean antiAliasing)
>> 
>> Constructor add:
>> public Scene(Parent root, double width, double height, boolean
>> depthBuffer, SceneAntiAliasing antiAliasing)
>> public SubScene(Parent root, double width, double height, boolean
>> depthBuffer, SceneAntiAliasing antiAliasing)
>> 
>> Note:The antiAliasing argument will be used if the underlying graphics
>> driver has anti-aliasing support.
>> 
>> Where SceneAntiAliasing is an enum with the following entries at the
>> moment:
>> 
>> public enum SceneAntiAliasing {
>>    DISABLED, // disables anti-aliasing
>>    DEFAULT, // enables anti-aliasing using a default system setting
>> available that balances speed and quality
>>    FASTEST, // enables anti-aliasing using minimum system setting
>> available that results in better frame rate
>>    NICEST // enables anti-aliasing using maximum system setting available
>> that results in best visual quality
>> }
>> 
>> Thanks,
>> - Chien
>>