CLN: .value -> ._values outside of core/
pandas.plotting I left alone
+1 on this, I agree we should not use `.values` at all; agree that using ``np.asarray()`` when we actually need a numpy arra (or .to_numpy()) explicitly is nice too. In an ideal world, we *never* should use `.values` and we can then *maybe* deprecate on the user side. @jbrockmendel I am fine with the general rule of not using `.values`. Although I think in cases where we explicitly need a numpy array, we should rather use `asarray` / `to_numpy`.

But there are still some relevant inline comments @jorisvandenbossche updated the one clearly-actionable inline comment.  can you point out which other comments you have in mind?  (or "resolve conversation" for the complement) Added new comments to those that are IMO still actionable, and resolved the others thanks