Alexander Plavin

Results 348 comments of Alexander Plavin

Well, unfortunate that this change is considered breaking instead of a bugfix: changing from objectively incorrect to correct behavior. Supporting left/right inverses is orthogonal, because this PR is only concerned...

Yes I think this should be merged, as a bugfix.

I also looked at other functions and fixed their inverse domains. It's quite hard to be sure that no edgecase is missed there! (most definitely some still remain) Nice that...

Sorry, I didn't imply any specific numbering of the next release. No matter the version number, this PR is still fundamentally a fix making behavior correct, following the function semantics...

Not that I have a lot of hope, but... Is there a chance this bugfix could be accepted anytime soon? I've stumbled across InverseFunctions silently doing a wrong thing several...

sorry, almost forgot about it!

Indeed, there were quite some code changes during this year :) Not sure what exactly went wrong here and caused this delay, even the original feature was accepted faster than...

Would make sense for `mean` as well. Currently, empty `mean` returns `NaN` for numbers and throws for non-numbers. Btw, `sum` lets you specify the default instead of zero, but the...

> Yes, but at least it does not throw an error in standard case Would be weird to have `default` in `median`, but not in `mean`. And I also wanted...

For `mean`, `NaN` seems the only right default choice, considering that `mean(x) = sum(x) / length(x)` and `0.0 / 0` is `NaN`. However, if these functions all returned `nothing` for...