Fix test() function so that it can distinguish NA from NaN #2365

Activity

1. st-pasha

   added this to the v1.10.6This milestone has been deleted milestone on Sep 15, 2017
2. st-pasha

   assigned

   mattdowle

   on Sep 15, 2017
3. mattdowle

   removed this from the v1.10.6This milestone has been deleted milestone on Sep 16, 2017
4. mattdowle commented on Sep 16, 2017

   

   mattdowle

   on Sep 16, 2017

   Member

   More actions

   I know this was my suggestion to change test(), but thinking about it, have now just added identical() into the test() calls in this rare case (PR #2363) where we care about NaN/NA differences. Changing test() to no longer use all.equal() (which considers NA and NaN equal) will be too much work for little benefit. Passing identical() to test() when we need that, should be fine.
5. mattdowle

   closed this as completedon Sep 16, 2017