Hi Christoph, Thank you for your response. The circ methods (circmean, circstd, and circvar) do not currently use the nan_policy keyword, and I wasn't able to find any pull requests addressing this on the GitHub page. I propose to add the nan_policy keyword option to circmean, circstd, and circvar. This keyword would then be passed to _circfuncs_commons, which currently tests the input samples and returns the samples as an array of angles in radians so that the circular statistics can be calculated. This makes it a sensible place to identify or omit any NaN (if desired). This way, if nan_policy is 'propagate', the circ methods would behave as they currently do, 'raise' would case _circfuncs_commons to throw an error, and 'omit' would remove all the NaN values before testing the sample size. Cheers, Angeline
participants (1)
-
Burrell, Angeline