Roy Smart
Roy Smart
I am having this exact same issue. Does anyone have a workaround for this problem, or any ideas for a patch that could resolve it?
I've opened Pull Request #13508 with my first attempt at implementing this functionality. I'm worried it's pretty inefficient / might lose precision since it converts times to Unix format before...
@mhvk, thanks for your suggestions, I've gone ahead and moved the logic to `astropy.time.Time.mean()` so that both the mean method and function work as expected. I will add your test...
@mhvk, I've added a test, `astropy.time.tests.test_methods.test_mean_precision()`, that tries to address your suggestions above.
@maxnoe, thanks for that documentation, I was about to go dig out my copy of Numerical Recipes to remind myself how to do this. Since numpy does not have a...
@mhvk and @taldcroft, the [tests are failing](https://github.com/astropy/astropy/runs/7550755821?check_suite_focus=true#step:6:1633) because `MaskedArray.mean()` doesn't support `where` (which sort of seems like a numpy bug to me, but I'm curious what you think). What do...
@mhvk, thanks for that implementation, that looks much better. I've updated the PR with those changes.
Does anyone know why `astropy.tables.tests.test_info.test_table_info_stats()` [fails during wheel building](https://github.com/astropy/astropy/runs/7616387347?check_suite_focus=true#step:4:3391) but [passes the normal CI](https://github.com/astropy/astropy/runs/7616392156?check_suite_focus=true#step:6:1125)?
I was trying to figure out how to make an unmasked `Time` from a masked one. Is there an easy way to do that?
Ok, just pushed a commit that tries to address combining `where` and `MaskedArray`. Made everything a little simpler which is nice :)