Behdad Esfahbod
Behdad Esfahbod
> > I just generated one with `momentRR` for Polar second moment, and the generator wasn't smart enough to reuse X^2 + Y^2 code for it. Which is weird... >...
Sorry for asking cosmetic changes, but please don't change the statisticsPen `area`; we don't want to change the API there.
> Sorry for asking cosmetic changes, but please don't change the statisticsPen `area`; we don't want to change the API there. Weird. You're not. I don't see why you pushed...
Oh... Damn. We have a problem. I had forgotten that `StatisticsPen` subclasses `MomentsPen`. So we *are* breaking API. Shoot.
Change of tactic. What do you think about reverting this back, and adding properties to StatisticsPen with the new names? Can also add the Polar 2nd moment there.
> re-adding properties to restore the old names, while keeping the new names internally, might be the smoother way for doing the higher-moments junk. What's the plan for higher moments...
> Right; I was thinking separate pen for sure. But it just seems preferable to have the two generated pens be consistent with each other on how they're generated &...
Okay then we just will wait for their PR or shutdown notice...
I think we should allow any UTF-8-encoded text as part of comments.
Sgtm.