[
https://issues.apache.org/jira/browse/MATH-1290?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15213268#comment-15213268
]
Gilles commented on MATH-1290:
------------------------------
Code available in branch "feature-MATH-1290".
Small remarks about the Javadoc (as a matter of personal taste):
* I don't think that the type of argument should be repeated in the "@param"
description (as it's shown in the method declaration).
* Whenever there is a math formula, we've agreed to use LaTeX (through
"MathJax") rather than HTML (which becomes quickly illegible, for those who
read the Javadoc in the source code).
* I much prefer that the "@param" description starts on the same line. In many
cases it doubles the number of comment lines for no obvious benefit (IMHO).
> Additions to Complex Utils; creation of conversion methods between Complex[],
> double[], float[], and interleaved arrays
> -----------------------------------------------------------------------------------------------------------------------
>
> Key: MATH-1290
> URL: https://issues.apache.org/jira/browse/MATH-1290
> Project: Commons Math
> Issue Type: Improvement
> Affects Versions: 3.5
> Reporter: Eric Barnhill
> Priority: Minor
> Labels: newbie, patch
> Fix For: 4.0
>
> Attachments: ComplexUtils.java, ComplexUtils.java,
> ComplexUtilsTest.java, ComplexUtilsTest.java, ComplexUtilsTest.java,
> LaguerreSolver.java
>
> Original Estimate: 24h
> Remaining Estimate: 24h
>
> I propose to add several new methods to the ComplexUtils class enabling easy
> conversion between Complex[] arrays, real or imaginary double[] or float[]
> arrays, and interleaved complex double[] or float[] arrays. The last two in
> particular are beneficial to have for OpenCL implementations and the
> JTransforms library.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
