Skip to content

14350 #18380

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

Merged
merged 2 commits into from
Nov 20, 2017
Merged

14350 #18380

merged 2 commits into from
Nov 20, 2017

Conversation

mpenkov
Copy link
Contributor

@mpenkov mpenkov commented Nov 20, 2017

@codecov
Copy link

codecov bot commented Nov 20, 2017

Codecov Report

Merging #18380 into master will increase coverage by 51.72%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           master   #18380       +/-   ##
===========================================
+ Coverage   39.61%   91.34%   +51.72%     
===========================================
  Files         164      164               
  Lines       49799    49799               
===========================================
+ Hits        19728    45487    +25759     
+ Misses      30071     4312    -25759
Flag Coverage Δ
#multiple 89.14% <ø> (?)
#single 39.55% <ø> (-0.07%) ⬇️
Impacted Files Coverage Δ
pandas/io/gbq.py 25% <0%> (-58.34%) ⬇️
pandas/core/computation/pytables.py 92.37% <0%> (+0.3%) ⬆️
pandas/io/pytables.py 92.84% <0%> (+0.97%) ⬆️
pandas/core/config_init.py 98.34% <0%> (+4.95%) ⬆️
pandas/io/formats/console.py 69.69% <0%> (+6.06%) ⬆️
pandas/compat/__init__.py 58.1% <0%> (+8.1%) ⬆️
pandas/core/reshape/util.py 90.32% <0%> (+12.9%) ⬆️
pandas/core/computation/common.py 85.71% <0%> (+14.28%) ⬆️
pandas/compat/numpy/__init__.py 93.93% <0%> (+18.18%) ⬆️
pandas/core/indexes/api.py 98.78% <0%> (+19.51%) ⬆️
... and 113 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1915ffc...9777771. Read the comment docs.

@jreback jreback added the Error Reporting Incorrect or improved errors from pandas label Nov 20, 2017
@jreback jreback added this to the 0.22.0 milestone Nov 20, 2017
@jreback jreback merged commit f85cdfd into pandas-dev:master Nov 20, 2017
@jreback
Copy link
Contributor

jreback commented Nov 20, 2017

thanks!

@mpenkov mpenkov deleted the 14350 branch November 20, 2017 11:11
@jorisvandenbossche
Copy link
Member

@mpenkov in general, can you try to give PRs a more informative title ?

@@ -26,6 +26,7 @@ Other Enhancements
- :func:`pandas.tseries.frequencies.to_offset` now accepts leading '+' signs e.g. '+1h'. (:issue:`18171`)
- :class:`pandas.io.formats.style.Styler` now has method ``hide_index()`` to determine whether the index will be rendered in ouptut (:issue:`14194`)
- :class:`pandas.io.formats.style.Styler` now has method ``hide_columns()`` to determine whether columns will be hidden in output (:issue:`14194`)
- Improved wording of ValueError raised in :func:`Timestamp.tz_localize` function
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this mentioning tz_localize? Wasn't this about to_datetime ?

Copy link
Contributor Author

@mpenkov mpenkov Nov 20, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're right, I specified the wrong function name in whatsnew. I pushed an update just now. Do I need to create a new PR for it?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hah i merged already
i’ll fix

jreback added a commit that referenced this pull request Nov 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Error Reporting Incorrect or improved errors from pandas
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DOC/ERR: update error message / doc-string for to_datetime with non-convertible object and unit kw
3 participants