Skip to content
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

centralize and de-privatize month/day name constants #18728

Merged
merged 4 commits into from
Dec 12, 2017

Conversation

jbrockmendel
Copy link
Member

  • closes #xxxx
  • tests added / passed
  • passes git diff upstream/master -u -- "*.py" | flake8 --diff
  • whatsnew entry

@jbrockmendel
Copy link
Member Author

I've got some debugging to do here and in two other places. In the interim, there's a bugfix in #18376 that I'm really eager to get closed.

@codecov
Copy link

codecov bot commented Dec 11, 2017

Codecov Report

Merging #18728 into master will decrease coverage by 0.01%.
The diff coverage is 90.47%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #18728      +/-   ##
==========================================
- Coverage   91.61%   91.59%   -0.02%     
==========================================
  Files         153      153              
  Lines       51339    51342       +3     
==========================================
- Hits        47034    47028       -6     
- Misses       4305     4314       +9
Flag Coverage Δ
#multiple 89.46% <90.47%> (ø) ⬆️
#single 40.74% <38.09%> (-0.12%) ⬇️
Impacted Files Coverage Δ
pandas/tseries/frequencies.py 94.04% <100%> (+0.02%) ⬆️
pandas/tseries/offsets.py 96.95% <88.23%> (ø) ⬆️
pandas/io/gbq.py 25% <0%> (-58.34%) ⬇️
pandas/core/frame.py 97.81% <0%> (-0.1%) ⬇️

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 2aa4aa9...de8dc13. Read the comment docs.

@codecov
Copy link

codecov bot commented Dec 11, 2017

Codecov Report

Merging #18728 into master will decrease coverage by 0.01%.
The diff coverage is 90.47%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #18728      +/-   ##
==========================================
- Coverage   91.61%   91.59%   -0.02%     
==========================================
  Files         153      153              
  Lines       51361    51364       +3     
==========================================
- Hits        47053    47047       -6     
- Misses       4308     4317       +9
Flag Coverage Δ
#multiple 89.45% <90.47%> (ø) ⬆️
#single 40.74% <38.09%> (-0.12%) ⬇️
Impacted Files Coverage Δ
pandas/tseries/frequencies.py 94.04% <100%> (+0.02%) ⬆️
pandas/tseries/offsets.py 96.95% <88.23%> (ø) ⬆️
pandas/io/gbq.py 25% <0%> (-58.34%) ⬇️
pandas/core/frame.py 97.81% <0%> (-0.1%) ⬇️

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 4091f64...a64a31f. Read the comment docs.

@jbrockmendel
Copy link
Member Author

circleCI is a timeout, appveyor looks unrelated, goes through fine locally:

___ TestDataFrameGroupByPlots.test_series_groupby_plotting_nominally_works ____
[gw0] win32 -- Python 3.6.3 C:\Miniconda3_64\envs\pandas\python.exe
self = <pandas.tests.plotting.test_groupby.TestDataFrameGroupByPlots object at 0x000000EA3231CCC0>
    def test_series_groupby_plotting_nominally_works(self):
        n = 10
        weight = Series(np.random.normal(166, 20, size=n))
        height = Series(np.random.normal(60, 10, size=n))
        with tm.RNGContext(42):
            gender = np.random.choice(['male', 'female'], size=n)
    
>       weight.groupby(gender).plot()
pandas\tests\plotting\test_groupby.py:25: 

@jreback jreback added Clean Frequency DateOffsets Datetime Datetime data dtype labels Dec 12, 2017
@mroeschke
Copy link
Member

Just a heads up, there are also some day names hardcoded here and here as well.

Linking #18164 as reference for myself.

@jreback jreback added this to the 0.22.0 milestone Dec 12, 2017
@jreback jreback merged commit 96439fb into pandas-dev:master Dec 12, 2017
@jreback
Copy link
Contributor

jreback commented Dec 12, 2017

@jbrockmendel pls add @mroeschke suggestions to the list, whomever wants can do a PR to centralize those as well!

@jbrockmendel jbrockmendel deleted the tslibs-constants branch January 23, 2018 04:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Clean Datetime Datetime data dtype Frequency DateOffsets
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants