-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
Unexpected behaviour while dropping #18287
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
@jorisvandenbossche how about being able to drop with boolean mask? Is there any discussion going on about it? |
I am not sure we had such a discussion, but I personally think we should not go that way: 1) to not complicate what |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Code Sample, a copy-pastable example if possible
Problem description
I see what is going here,
ser == 'c'
results in a list of True/False (meaning 0 and 1), so it drops the 0 and 1 indices. However, this is a completely unexpected result. This should at least raise an exception. I would be happy with an error, but boolean mask support would result in a much more cleaner and simpler syntax than:Expected Output
Raising an exception or dropping by boolean mask:
Output of
pd.show_versions()
INSTALLED VERSIONS
commit: None
pandas: 0.19.2
nose: None
pip: 9.0.1
setuptools: 36.6.0
Cython: None
numpy: 1.11.0
scipy: 0.18.1
statsmodels: None
xarray: None
IPython: 6.2.1
sphinx: 1.6.4
patsy: None
dateutil: 2.6.1
pytz: 2017.2
blosc: None
bottleneck: None
tables: None
numexpr: None
matplotlib: 2.1.0
openpyxl: None
xlrd: None
xlwt: None
xlsxwriter: None
lxml: None
bs4: None
html5lib: None
httplib2: None
apiclient: None
sqlalchemy: None
pymysql: None
psycopg2: None
jinja2: 2.9.6
boto: None
pandas_datareader: None
The text was updated successfully, but these errors were encountered: