Message194503
> My only objection is to having a class xyz such that isinstance(xyz(..),
> xyz) is false.
Yep. Use a set of functions (median, median_low); use an instance of a class as Alexander describes; use a single median function that takes an optional "method" parameter; create a statistics.median subpackage and put the various median functions in that. Any of those options are fairly standard, unsurprising, and could reasonably be defended.
But having `median` be a class whose `__new__` returns a float really *is* nonstandard and peculiar. There's just no need for such perversity in what should be a straightforward and uncomplicated module. Special cases aren't special enough to break the rules and all that. |
|
Date |
User |
Action |
Args |
2013-08-05 18:57:19 | mark.dickinson | set | recipients:
+ mark.dickinson, gregory.p.smith, ronaldoussoren, belopolsky, pitrou, agthorr, christian.heimes, stutzbach, steven.daprano, tshepang, vajrasky |
2013-08-05 18:57:19 | mark.dickinson | set | messageid: <1375729039.68.0.696618611669.issue18606@psf.upfronthosting.co.za> |
2013-08-05 18:57:19 | mark.dickinson | link | issue18606 messages |
2013-08-05 18:57:19 | mark.dickinson | create | |
|