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

DEPS: Drop Python 3.4 support #16303

Merged
merged 2 commits into from
May 16, 2017
Merged

Conversation

gfyoung
Copy link
Member

@gfyoung gfyoung commented May 9, 2017

Title is self-explanatory.

Closes #15251.

@TomAugspurger TomAugspurger added this to the 0.21.0 milestone May 9, 2017
@codecov
Copy link

codecov bot commented May 9, 2017

Codecov Report

Merging #16303 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #16303      +/-   ##
==========================================
- Coverage   90.39%   90.37%   -0.02%     
==========================================
  Files         161      161              
  Lines       50863    50863              
==========================================
- Hits        45977    45968       -9     
- Misses       4886     4895       +9
Flag Coverage Δ
#multiple 88.16% <ø> (ø) ⬆️
#single 40.33% <ø> (-0.11%) ⬇️
Impacted Files Coverage Δ
pandas/compat/__init__.py 61.77% <ø> (ø) ⬆️
pandas/io/gbq.py 25% <0%> (-58.34%) ⬇️
pandas/core/frame.py 97.59% <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 ce4eef3...a186ba8. Read the comment docs.

@codecov
Copy link

codecov bot commented May 9, 2017

Codecov Report

Merging #16303 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #16303      +/-   ##
==========================================
- Coverage   90.38%   90.36%   -0.03%     
==========================================
  Files         161      161              
  Lines       50916    50916              
==========================================
- Hits        46021    46009      -12     
- Misses       4895     4907      +12
Flag Coverage Δ
#multiple 88.13% <ø> (-0.01%) ⬇️
#single 40.22% <ø> (-0.11%) ⬇️
Impacted Files Coverage Δ
pandas/compat/__init__.py 61.77% <ø> (ø) ⬆️
pandas/io/gbq.py 25% <0%> (-58.34%) ⬇️
pandas/core/common.py 90.68% <0%> (-0.35%) ⬇️
pandas/util/testing.py 80.56% <0%> (-0.2%) ⬇️
pandas/core/frame.py 97.68% <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 0ea0f25...9c33e44. Read the comment docs.

@jreback jreback added the Dependencies Required and optional dependencies label May 9, 2017
@jreback
Copy link
Contributor

jreback commented May 10, 2017

so we have a couple of circle ci jobs which test 3.4 move these to 3.6 i think. (they still hit a case which we need to test).

@gfyoung
Copy link
Member Author

gfyoung commented May 10, 2017

@jreback : Good catch. Copied and pasted the requirements-3.6* files to requirements-3.6_SLOW* files and also removed 3.4 from the tox.ini file (git diff is not very helpful, hence why I'm explaining my changes here).

@gfyoung gfyoung force-pushed the python-3.4-drop branch 2 times, most recently from 78edc13 to 9f9389d Compare May 10, 2017 17:11
@gfyoung
Copy link
Member Author

gfyoung commented May 10, 2017

@jreback : Weird locale failure on Circle here. Not sure what is wrong with the test files.

@gfyoung gfyoung force-pushed the python-3.4-drop branch 2 times, most recently from c628c2d to 1b3ec7c Compare May 11, 2017 21:21
@gfyoung
Copy link
Member Author

gfyoung commented May 12, 2017

@jreback : Everything is green here, but I presume you are waiting to merge this until AFTER 0.20.2 is released into the wild (correct me if I'm wrong).

@jreback
Copy link
Contributor

jreback commented May 12, 2017

@gfyoung no, we can merge things for 0.21.0 now. @jorisvandenbossche had an objection. I actually have no problem with this. yes its not hard to support. But 3.4 is not very well supported by conda anymore, no point in us holding out the torch.

@gfyoung
Copy link
Member Author

gfyoung commented May 12, 2017

@jreback : So is this good to go then?

@@ -14,7 +14,7 @@ html5lib
jinja2
sqlalchemy
pymysql
feather-format
# feather-format (not available on defaults ATM)
Copy link
Contributor

Choose a reason for hiding this comment

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

no, the 3.6 run specially uses conda-forge (this is the only run that does this)

Copy link
Member Author

Choose a reason for hiding this comment

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

So just change the comment?

Copy link
Contributor

Choose a reason for hiding this comment

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

revert the change
you shouldn't be changing this file
it's for the travis build

Copy link
Member Author

Choose a reason for hiding this comment

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

Circle broke when i left the file as is. Couldn't install feather.

Copy link
Contributor

Choose a reason for hiding this comment

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

u need a separate file (and a different job tag)
also ok to just have 3 circle jobs

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, okay. That's works. Will do then.

@gfyoung
Copy link
Member Author

gfyoung commented May 13, 2017

@jreback : Made the requested changes, and everything is still green. PTAL.

circle.yml Outdated
@@ -23,9 +23,9 @@ dependencies:
0)
sudo apt-get install language-pack-it && ./ci/install_circle.sh JOB="2.7_COMPAT" LOCALE_OVERRIDE="it_IT.UTF-8" ;;
1)
sudo apt-get install language-pack-zh-hans && ./ci/install_circle.sh JOB="3.4_SLOW" LOCALE_OVERRIDE="zh_CN.UTF-8" ;;
sudo apt-get install language-pack-zh-hans && ./ci/install_circle.sh JOB="3.6_SLOW" LOCALE_OVERRIDE="zh_CN.UTF-8" ;;
Copy link
Contributor

Choose a reason for hiding this comment

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

actuall let's name these

3.6_LOCALE_SLOW
3.6_LOCALE

I think it more consistent

Copy link
Member Author

Choose a reason for hiding this comment

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

Sounds good. Done.

@gfyoung
Copy link
Member Author

gfyoung commented May 13, 2017

@jreback : Made the requested changes, and everything is still green. PTAL.

@@ -0,0 +1,6 @@
python=3.6*
Copy link
Contributor

Choose a reason for hiding this comment

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

seems you added a 3.6_SLOW, pls remove.

Copy link
Member Author

Choose a reason for hiding this comment

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

Oops. Done.

@gfyoung
Copy link
Member Author

gfyoung commented May 15, 2017

@jreback : Made the requested changes, and everything is still green. PTAL.

@jreback jreback merged commit 2ba7f5f into pandas-dev:master May 16, 2017
@jreback
Copy link
Contributor

jreback commented May 16, 2017

thanks!

@gfyoung gfyoung deleted the python-3.4-drop branch May 16, 2017 07:15
pawroman added a commit to pawroman/pandas that referenced this pull request May 18, 2017
* upstream/master: (48 commits)
  BUG: Categorical comparison with unordered (pandas-dev#16339)
  ENH: Adding 'protocol' parameter to 'to_pickle'.
  PERF: improve MultiIndex get_loc performance (pandas-dev#16346)
  TST: remove pandas-datareader xfail as 0.4.0 works (pandas-dev#16374)
  TST: followup to pandas-dev#16364, catch errstate warnings (pandas-dev#16373)
  DOC: new oauth token
  TST: Add test for clip-na (pandas-dev#16369)
  ENH: Draft metadata specification doc for Apache Parquet (pandas-dev#16315)
  MAINT: Add .iml to .gitignore (pandas-dev#16368)
  BUG/API: Categorical constructor scalar categories (pandas-dev#16340)
  ENH: Provide dict object for to_dict() pandas-dev#16122 (pandas-dev#16220)
  PERF: improved clip performance (pandas-dev#16364)
  DOC: try new token for docs
  DOC: try with new secure token
  DOC: add developer section to the docs
  DEPS: Drop Python 3.4 support (pandas-dev#16303)
  DOC: remove credential helper
  DOC: force fetch on build docs
  DOC: redo dev docs access token
  DOC: add dataframe construction in merge_asof example (pandas-dev#16348)
  ...
pcluo pushed a commit to pcluo/pandas that referenced this pull request May 22, 2017
* DEPS: Drop Python 3.4 support

Closes pandas-devgh-15251.

* TST: Patch locale failure on Circle
stangirala pushed a commit to stangirala/pandas that referenced this pull request Jun 11, 2017
* DEPS: Drop Python 3.4 support

Closes pandas-devgh-15251.

* TST: Patch locale failure on Circle
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dependencies Required and optional dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants