[
https://issues.apache.org/jira/browse/MATH-1132?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14045947#comment-14045947
]
Venkatesha Murthy TS commented on MATH-1132:
--------------------------------------------
I should have mentioned this earlier.
After discussing in Dev ML on NaNStrategy#Fixed issue (sub: [math] Usecase fpr
NaNStrategy#FIXED) where in the last thread of the mail it mentions about
creating a new ticket for documenting the changes. Hence this new ticket.
Sure Gilles, thanks for this suggestion and will continue the discussion on ML
on this using the same subject. However if i need to explain my proposition i
need to write up that bit of code. So wondering how i could communicate that
(Do i put it in my local git and give a link)
So in this case , please help me understand :
a) If NanTransformer approves then this ticket is the first to move on
b) Next we need to open another ticket for including this refactor for
percentile?
Thanks for suggestions/corrections. I will most certainly take this up.
Regards
Venkat
> NaN transformation based on NaNStrategy
> ---------------------------------------
>
> Key: MATH-1132
> URL: https://issues.apache.org/jira/browse/MATH-1132
> Project: Commons Math
> Issue Type: Improvement
> Affects Versions: 3.3
> Reporter: Venkatesha Murthy TS
> Fix For: 3.4
>
> Attachments: math-1132.patch
>
>
> The NaNs in a double array may need to be transformed to either +/- Infinity
> or removed before processing them(say such as in percentile).
> So basically from documenting angle; we will need to action as follows:
> Whenever a double quantity is a NaN (say in an array); based on the
> NanStrategy; it needs to be treated as follows before processing:
> a) MAXIMAL -> replace with +infinity
> b) MINIMAL -> replace with -Inifinity
> c) REMOVED -> just remove it
> d) FIXED -> Leave it as it is
> e) FAILED -> throw a NaN exception
> Can this documentation be specifically added to NaNStrategy and perhaps a
> substitution value also can be added to enforce the documentation.
> In addition i will also submit a patch for a simple NaNTransformer
> implementation.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
