Skip to content

Conversation

@srf94
Copy link

@srf94 srf94 commented Jun 26, 2019

@pep8speaks
Copy link

pep8speaks commented Jun 26, 2019

Hello @srf94! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2019-06-26 19:58:19 UTC
@srf94 srf94 force-pushed the update_doc_assertion_raised branch from c393e8e to d3218c7 Compare June 26, 2019 19:58
@codecov
Copy link

codecov bot commented Jun 26, 2019

Codecov Report

Merging #27067 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@ ## master #27067 +/- ## ========================================== - Coverage 92.04% 92.03% -0.01%  ========================================== Files 180 180 Lines 50714 50714 ========================================== - Hits 46680 46676 -4  - Misses 4034 4038 +4
Flag Coverage Δ
#multiple 90.67% <ø> (ø) ⬆️
#single 41.88% <ø> (-0.09%) ⬇️
Impacted Files Coverage Δ
pandas/core/frame.py 96.89% <ø> (-0.12%) ⬇️
pandas/io/gbq.py 88.88% <0%> (-11.12%) ⬇️

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 d94146c...d3218c7. Read the comment docs.

1 similar comment
@codecov
Copy link

codecov bot commented Jun 26, 2019

Codecov Report

Merging #27067 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@ ## master #27067 +/- ## ========================================== - Coverage 92.04% 92.03% -0.01%  ========================================== Files 180 180 Lines 50714 50714 ========================================== - Hits 46680 46676 -4  - Misses 4034 4038 +4
Flag Coverage Δ
#multiple 90.67% <ø> (ø) ⬆️
#single 41.88% <ø> (-0.09%) ⬇️
Impacted Files Coverage Δ
pandas/core/frame.py 96.89% <ø> (-0.12%) ⬇️
pandas/io/gbq.py 88.88% <0%> (-11.12%) ⬇️

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 d94146c...d3218c7. Read the comment docs.

Copy link
Member

@WillAyd WillAyd left a comment

Choose a reason for hiding this comment

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

Can you check if we have test coverage for this scenario?

@WillAyd WillAyd added DataFrame DataFrame data structure Docs Error Reporting Incorrect or improved errors from pandas labels Jun 27, 2019
@jorisvandenbossche
Copy link
Member

jorisvandenbossche commented Jun 27, 2019

I think that we indeed first need to pin down the behaviour in code and tests. As it does not seem to always give an error message. For example:

In [24]: data = {1: ["foo"], 2: ["bar"]} In [25]: pd.DataFrame.from_records(data, columns=['a', 'b']) Out[25]: Empty DataFrame Columns: [a, b] Index: [] 

This is an example from the tests:

data = {1: ['foo'], 2: ['bar']}
result = DataFrame.from_records(data, columns=['a', 'b'])
exp = DataFrame(data, columns=['a', 'b'])
tm.assert_frame_equal(result, exp)

@jorisvandenbossche
Copy link
Member

Also, the sentence you are changing is about the case that your data already have names. While the use case in the reported issue is when the data have no names.

@WillAyd
Copy link
Member

WillAyd commented Aug 28, 2019

This looks stale so closing for now but ping if you'd like to pick back up

@WillAyd WillAyd closed this Aug 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DataFrame DataFrame data structure Docs Error Reporting Incorrect or improved errors from pandas

4 participants