- From: Tavmjong Bah <tavmjong@free.fr>
- Date: Sun, 14 Nov 2010 11:23:32 +0100
- To: Patrick Dengler <patd@microsoft.com>
- Cc: SVG Working Group WG <public-svg-wg@w3.org>
On Fri, 2010-11-12 at 00:14 +0000, Patrick Dengler wrote:
> We have been looking at the filters chapter and believe that there
> would be a benefit for setting min/max for some of the attributes. We
> are recommending these as currently, without these bounds, most
> browsers would likely peg the CPU so badly, it would lead it to be
> inoperable, or are reasonable limits a developer would want or a user
> could perceive. We'd like to see these expressed in a future version
> of the filters specification, but would like to see it documented.
One should be a bit careful in setting limits as sometimes what are
useful values aren't initially apparent. I've fought this battle a few
times with Inkscape where someone changes the limit in the Inkscape
interface and breaks one of my examples. Having said that, I don't see
any real problems with the proposed limits.
> feConvolveMatrix: Order (Kernel Units)
> Min : (1, 1)
> Max: (10, 10)
Seems reasonable other than odd orders are probably more useful than
even orders so an odd maximum might be more appropriate (9 or 11?).
Inkscape built-in filters only use an order of 3. I've used 5 in some
examples.
> feConvolveMatrix, feSpecularlighting , feDiffuseLighting: Kernel Unit Length
> Min: 0.1
> Max: 10
Inkscape built-in filters don't use KernelUnitLength at the moment. The
spec mentions that pixel alignment can result in faster performance so
perhaps a maximum that is a power of 2 is appropriate (16?).
> feMorphology: Radius
> Max: 150
Range in Inkscape built-in filters is 0.65 to 10.
> feTurblulance: numOctaves
> Max: 10
Range in Inkscape built-in filters is 1 to 10. The use of 10 maybe the
result of that being the maximum value allowed in the Inkscape Filter
Dialog. I decreased the value to 5 in all the filters where it was used
and saw only minor differences. Ten should be a safe maximum.
Tav
Received on Sunday, 14 November 2010 10:24:09 UTC