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

Rename DataColumn.to_pandas to DataColumn.to_series #311

Merged
merged 3 commits into from
Oct 24, 2020
Merged

Conversation

thehomebrewnerd
Copy link
Contributor

Renames the DataColumn method to_pandas to to_series to better reflect return types when using non-pandas dataframes for input. Also updated documentation where this was mentioned related to columns.

@codecov
Copy link

codecov bot commented Oct 23, 2020

Codecov Report

Merging #311 into main will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##              main      #311   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           22        22           
  Lines         2550      2550           
=========================================
  Hits          2550      2550           
Impacted Files Coverage Δ
woodwork/data_column.py 100.00% <100.00%> (ø)
woodwork/tests/data_column/test_data_column.py 100.00% <100.00%> (ø)
woodwork/tests/data_table/test_datatable.py 100.00% <100.00%> (ø)
woodwork/tests/testing_utils/data_table_utils.py 100.00% <100.00%> (ø)

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 b77e6ed...09e20bc. Read the comment docs.

* Documentation Changes
* Add conda install instructions (:pr:`305`, :pr:`309`)
* Testing Changes

Thanks to the following people for contributing to this release:
:user:`ctduffy`, :user:`gsheni`, :
Thanks to the following people for contributing to this release:
Copy link
Contributor

Choose a reason for hiding this comment

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

We should add that this is a breaking change? Users will have to rename their function call?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes - we should. I'll update.

@gsheni gsheni merged commit b16eed4 into main Oct 24, 2020
@gsheni gsheni deleted the column-to-series branch October 24, 2020 01:52
@gsheni gsheni mentioned this pull request Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rename to_pandas to be more generic on DataColumn - to_series
2 participants