On 7/13/06, Remi Arntzen <[EMAIL PROTECTED]> wrote:
On 7/4/06, Phil Steitz <[EMAIL PROTECTED]> wrote:
> Hi Remy,
>
> The test coverage is still not where we would like to have it and we
> may want to refine/revise the API.  Please have a look and if you have
> suggestions for improvement or are willing to work on adding more test
> cases / validation with other packages, thanks in advance.
>

Well now that you mention it the current FastFourierTransformer class
is not public for me to use.  The transform method is public, but it
is also non-static and thus requires an instantiation of the class.
However the class is non-instantiable, because the constructor for the
class only has package level access.

I want to start work on a multi dimensional extension to this class,
but it would be somewhat easier if the constructor was made public.

Fixed in svn.  Sorry for the latency on this.  In the future, to make
sure these things don't fall through the cracks, its best to open JIRA
tickets.

Along the same lines, an extension to the API could use a utility
class that contains only static versions of the corresponding methods
in the FastFourierTransformer class.  The original class could be
remade with only static methods, but I believe it would be far quicker
to make a FourierUtils ( :-) ) class or something.


+1 to the Utils class along the lines of StatUtils in the stat package.

Thanks for reporting the access problem.

Phil

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to