BUG: Can't store callables using __setitem__ #13516
Conversation
pandas/core/generic.py
Outdated
| Whether to raise on invalid data types (e.g. trying to where on | ||
| strings) | ||
| apply_other : boolean, default True | ||
| If False, other will be stored directly rather than applied to |
There was a problem hiding this comment.
I don't think we need another parameter
it's remove the callable check on other entirely
There was a problem hiding this comment.
That's fine with me; I left it in because @sinhrks wanted it for the case where "where" is called directly rather than through setitem.
There was a problem hiding this comment.
yeah I think we just shouldn't act on other at all.
There was a problem hiding this comment.
How about adding internal _where which doesn't apply callable at all? where can use it after applying callable.
I think applying callable to other in where is sometimes useful (described in #13299).
jreback
added
Indexing
|
yeah that's a better soln |
|
|
||
| @Appender(_shared_docs['where'] % dict(_shared_doc_kwargs, cond="True")) | ||
| def where(self, cond, other=np.nan, inplace=False, axis=None, level=None, | ||
| def _where(self, cond, other=np.nan, inplace=False, axis=None, level=None, |
There was a problem hiding this comment.
Can you add a docstring to this one explaining it's purpose?
|
lgtm. |
|
lgtm. @jorisvandenbossche |
doc/source/whatsnew/v0.18.2.txt
Outdated
| - Bug in ``pd.read_hdf()`` where attempting to load an HDF file with a single dataset, that had one or more categorical columns, failed unless the key argument was set to the name of the dataset. (:issue:`13231`) | ||
| - Bug in ``.rolling()`` that allowed a negative integer window in contruction of the ``Rolling()`` object, but would later fail on aggregation (:issue:`13383`) | ||
|
|
||
| - Bug in ``__setitem__`` causing a callable rhs to be applied rather than stored (:issue:`13299`) |
There was a problem hiding this comment.
actually maybe put this in API changes to make it a bit more prominent.
Current coverage is 84.34%@@ master #13516 diff @@
==========================================
Files 138 138
Lines 51107 51113 +6
Methods 0 0
Messages 0 0
Branches 0 0
==========================================
+ Hits 43103 43109 +6
Misses 8004 8004
Partials 0 0
|
|
Thanks a lot! |
5 participants
git diff upstream/master | flake8 --diff