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

CLN: Removed outtype in DataFrame.to_dict #13627

Merged

Conversation

gfyoung
Copy link
Member

@gfyoung gfyoung commented Jul 12, 2016

Follows up from #8486 in 0.15.0 by removing outtype in DataFrame.to_dict()

Fortunately or unfortunately, no tests were written then to test the deprecation, so there was nothing to remove from the test suite this time around.

@@ -443,6 +443,7 @@ Removal of prior version deprecations/changes
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

- ``DataFrame.to_csv()`` has dropped the ``engine`` parameter, as was deprecated in 0.17.1 (:issue:`11274`, :issue:`13419`)
- ``DataFrame.to_dict()`` has dropped the ``outtype`` parameter in favor of ``orient`` (:issue:``)
Copy link
Contributor

Choose a reason for hiding this comment

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

put this issue number then

Copy link
Contributor

Choose a reason for hiding this comment

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

and the issue you reference in the issue

Copy link
Member Author

Choose a reason for hiding this comment

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

Done.

@jreback jreback added the Deprecate Functionality to remove in pandas label Jul 12, 2016
@jreback jreback added this to the 0.19.0 milestone Jul 12, 2016
@codecov-io
Copy link

codecov-io commented Jul 12, 2016

Current coverage is 84.33%

Merging #13627 into master will decrease coverage by <.01%

@@             master     #13627   diff @@
==========================================
  Files           138        138          
  Lines         51100      51099     -1   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits          43097      43096     -1   
  Misses         8003       8003          
  Partials          0          0          

Powered by Codecov. Last updated by 65849d3...1b5099d

@gfyoung
Copy link
Member Author

gfyoung commented Jul 12, 2016

@jreback : Made the requested doc change (no tests need as a result hence the [ci skip], so ready to merge if there are no other concerns.

@jorisvandenbossche jorisvandenbossche merged commit 7c357d2 into pandas-dev:master Jul 12, 2016
@jorisvandenbossche
Copy link
Member

@gfyoung Thanks!

@gfyoung gfyoung deleted the df-to-dict-outtype-remove branch July 13, 2016 01:46
nateGeorge pushed a commit to nateGeorge/pandas that referenced this pull request Aug 15, 2016
Follows up from pandas-dev#8486 in 0.15.0 by removing outtype in DataFrame.to_dict()
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Deprecate Functionality to remove in pandas
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants