BrandonFors/pandas-github-issues huggingface dataset:
ENH: Option to return -inf/inf as lower/upper bound of quantiles, see…
State: closed

Add `bounded` keyword arg to `quantile` and `qcut` functions to enable unbounded quantiles with lower/upper bounds of -inf/inf.

- [x] closes #17282
- [ ] tests added / passed
- [x] passes `git diff upstream/master -u -- "*.py" | flake8 --diff`
- [x] whatsnew entry


Labels:

Comments:
Hello @dberenbaum! Thanks for submitting the PR.



 - In the file [`pandas/core/reshape/tile.py`](https://github.com/pandas-dev/pandas/blob/48b5f5c6298ff30a9184f1ae21096d5a017dbfb1/pandas/core/reshape/tile.py), following are the PEP8 issues :

> [Line 241:70](https://github.com/pandas-dev/pandas/blob/48b5f5c6298ff30a9184f1ae21096d5a017dbfb1/pandas/core/reshape/tile.py#L241): [W291](https://duckduckgo.com/?q=pep8%20W291) trailing whitespace
Remove unused cimports, fix #22067
State: closed

Fixes #22067, but no clear way to test it.

- [x] closes #22067


Labels:
name: Bug
name: Period
description: Period data type
name: Clean

Comments:
minor comments and pls rebase
Hello @jbrockmendel! Thanks for updating the PR.



Cheers ! There are no PEP8 issues in this Pull Request. :beers: 

##### Comment last updated on July 29, 2018 at 20:57 Hours UTC
# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22087?src=pr&el=h1) Report
> Merging [#22087](https://codecov.io/gh/pandas-dev/pandas/pull/22087?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/d30c4a0696d5fbdc3c7ce36a9b9b19224a557e09?src=pr&el=desc) will **not change** coverage.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/pandas-dev/pandas/pull/22087/graphs/tree.svg?height=150&width=650&src=pr&token=eZ4WkYLtcO)](https://codecov.io/gh/pandas-dev/pandas/pull/22087?src=pr&el=tree)

```diff
@@           Coverage Diff           @@
##           master   #22087   +/-   ##
=======================================
  Coverage   92.07%   92.07%           
=======================================
  Files         170      170           
  Lines       50690    50690           
=======================================
  Hits        46672    46672           
  Misses       4018     4018
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `90.48% <ø> (ø)` | :arrow_up: |
| #single | `42.31% <ø> (ø)` | :arrow_up: |


------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22087?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22087?src=pr&el=footer). Last update [d30c4a0...a1bff36](https://codecov.io/gh/pandas-dev/pandas/pull/22087?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

Ping
thanks!
DOC: clarify default behavior for categories ordering
State: closed

- [x] passes `git diff upstream/master -u -- "*.py" | flake8 --diff`

Just a doc clarification on default categories ordering, which can be pretty confusing when handling mixed dtypes (see also #22080 ).

Labels:
name: Docs
name: MultiIndex
name: Categorical
description: Categorical Data Type

Comments:
# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22081?src=pr&el=h1) Report
> Merging [#22081](https://codecov.io/gh/pandas-dev/pandas/pull/22081?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/26b3e7de56fdebaf1842b01d747a1a30a126c54a?src=pr&el=desc) will **not change** coverage.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/pandas-dev/pandas/pull/22081/graphs/tree.svg?src=pr&token=eZ4WkYLtcO&width=650&height=150)](https://codecov.io/gh/pandas-dev/pandas/pull/22081?src=pr&el=tree)

```diff
@@           Coverage Diff           @@
##           master   #22081   +/-   ##
=======================================
  Coverage   92.07%   92.07%           
=======================================
  Files         170      170           
  Lines       50688    50688           
=======================================
  Hits        46671    46671           
  Misses       4017     4017
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `90.48% <ø> (ø)` | :arrow_up: |
| #single | `42.31% <ø> (ø)` | :arrow_up: |

| [Impacted Files](https://codecov.io/gh/pandas-dev/pandas/pull/22081?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [pandas/core/arrays/categorical.py](https://codecov.io/gh/pandas-dev/pandas/pull/22081/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvYXJyYXlzL2NhdGVnb3JpY2FsLnB5) | `95.95% <ø> (ø)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22081?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22081?src=pr&el=footer). Last update [26b3e7d...ddb03e4](https://codecov.io/gh/pandas-dev/pandas/pull/22081?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

lgtm. @TomAugspurger @jorisvandenbossche wording look ok?
CLN/STYLE: Lint comprehensions
State: closed

- [x] closes #18464
- [x] tests added / passed
- [x] passes `git diff upstream/master -u -- "*.py" | flake8 --diff`

Using flake8-comprehensions (xref https://github.com/pandas-dev/pandas/issues/20588#issuecomment-405057018) to lint for unnecessary comprehensions. I just added this to the 2.7-travis build since it appears to be our linting build.

Labels:
name: Code Style
description: Code style, linting, code_checks

Comments:
Syntax error in one of the edits.  +1 on the concept.
Tons of flake complaints, may need to implement gradually.
Now it’s mostly upset about lambdas.  OTOH it’s reassuring to see mine isn’t the only PR that pytables is messing up.
Is there a predefined set of lint codes we ignore?

I am a little puzzled why these lint checks now started to fire; I figure it has something to do with the scope of checks when `--ignore` is specified.
> Is there a predefined set of lint codes we ignore?

in setup.cfg
Possibly for the while-were-at-it pile: 

#11954 checks for unnecessary backslashes (could this be done with just a grep?)
Enable checks for bare “except:” statements in a subset of files and gradually expand?
we have a fixed version of flake8 (an open issue about this too) - which doesn’t check bare excepts 
pls rebase
The travis failure is from the reoccuring `ResourceWarning` that appears in `test_dialect_conflict` 
thanks @mroeschke 

if you think need to open additional issues for the skipped pep codes, pls do. alternatively maybe just add some docs on them on why we need to keep them there (e.g. so next person isn't wondering why those particular codes).
TST: Parameterize more tests
State: closed

Title is self-explanatory.


Labels:
name: Refactor
description: Internal refactoring of code
name: Testing
description: pandas testing functions or related to the test suite

Comments:
# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22061?src=pr&el=h1) Report
> Merging [#22061](https://codecov.io/gh/pandas-dev/pandas/pull/22061?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/2c7c79779f1ef353d02446504d3b5f5c5cc68759?src=pr&el=desc) will **not change** coverage.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/pandas-dev/pandas/pull/22061/graphs/tree.svg?token=eZ4WkYLtcO&width=650&height=150&src=pr)](https://codecov.io/gh/pandas-dev/pandas/pull/22061?src=pr&el=tree)

```diff
@@           Coverage Diff           @@
##           master   #22061   +/-   ##
=======================================
  Coverage   92.02%   92.02%           
=======================================
  Files         170      170           
  Lines       50710    50710           
=======================================
  Hits        46664    46664           
  Misses       4046     4046
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `90.42% <ø> (ø)` | :arrow_up: |
| #single | `42.31% <ø> (ø)` | :arrow_up: |


------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22061?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22061?src=pr&el=footer). Last update [2c7c797...f2f5fb6](https://codecov.io/gh/pandas-dev/pandas/pull/22061?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

noice!
DOC: Reword doc for filepath_or_buffer in read_csv
State: closed

Shortens dtype description following colon and moves it to a more expanded parameter description.

Closes #22057.

cc @WillAyd 

Labels:
name: Docs
name: IO CSV
description: read_csv, to_csv

Comments:
Hello @gfyoung! Thanks for updating the PR.



Cheers ! There are no PEP8 issues in this Pull Request. :beers: 

##### Comment last updated on July 26, 2018 at 08:37 Hours UTC
# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22058?src=pr&el=h1) Report
> Merging [#22058](https://codecov.io/gh/pandas-dev/pandas/pull/22058?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/2c7c79779f1ef353d02446504d3b5f5c5cc68759?src=pr&el=desc) will **not change** coverage.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/pandas-dev/pandas/pull/22058/graphs/tree.svg?width=650&height=150&src=pr&token=eZ4WkYLtcO)](https://codecov.io/gh/pandas-dev/pandas/pull/22058?src=pr&el=tree)

```diff
@@           Coverage Diff           @@
##           master   #22058   +/-   ##
=======================================
  Coverage   92.02%   92.02%           
=======================================
  Files         170      170           
  Lines       50710    50710           
=======================================
  Hits        46664    46664           
  Misses       4046     4046
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `90.42% <ø> (ø)` | :arrow_up: |
| #single | `42.31% <ø> (ø)` | :arrow_up: |

| [Impacted Files](https://codecov.io/gh/pandas-dev/pandas/pull/22058?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [pandas/io/parsers.py](https://codecov.io/gh/pandas-dev/pandas/pull/22058/diff?src=pr&el=tree#diff-cGFuZGFzL2lvL3BhcnNlcnMucHk=) | `95.46% <ø> (ø)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22058?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22058?src=pr&el=footer). Last update [2c7c797...92cc578](https://codecov.io/gh/pandas-dev/pandas/pull/22058?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

@jreback @jorisvandenbossche : Made the requested changes, and all is green.  PTAL.
Thanks!
BUG: Matplotlib scatter datetime
State: closed

xref https://github.com/matplotlib/matplotlib/issues/11391

Labels:
name: Visualization
description: plotting

Comments:
# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22039?src=pr&el=h1) Report
> :exclamation: No coverage uploaded for pull request base (`master@4cf95ba`). [Click here to learn what that means](https://docs.codecov.io/docs/error-reference#section-missing-base-commit).
> The diff coverage is `100%`.

[![Impacted file tree graph](https://codecov.io/gh/pandas-dev/pandas/pull/22039/graphs/tree.svg?width=650&src=pr&token=eZ4WkYLtcO&height=150)](https://codecov.io/gh/pandas-dev/pandas/pull/22039?src=pr&el=tree)

```diff
@@            Coverage Diff            @@
##             master   #22039   +/-   ##
=========================================
  Coverage          ?      92%           
=========================================
  Files             ?      170           
  Lines             ?    50611           
  Branches          ?        0           
=========================================
  Hits              ?    46567           
  Misses            ?     4044           
  Partials          ?        0
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `90.41% <100%> (?)` | |
| #single | `42.19% <0%> (?)` | |

| [Impacted Files](https://codecov.io/gh/pandas-dev/pandas/pull/22039?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [pandas/plotting/\_converter.py](https://codecov.io/gh/pandas-dev/pandas/pull/22039/diff?src=pr&el=tree#diff-cGFuZGFzL3Bsb3R0aW5nL19jb252ZXJ0ZXIucHk=) | `63.88% <100%> (ø)` | |

------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22039?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22039?src=pr&el=footer). Last update [4cf95ba...0132fea](https://codecov.io/gh/pandas-dev/pandas/pull/22039?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

thanks @TomAugspurger
[wip] debug failures
State: closed


Labels:
name: Testing
description: pandas testing functions or related to the test suite

Comments:
# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22038?src=pr&el=h1) Report
> Merging [#22038](https://codecov.io/gh/pandas-dev/pandas/pull/22038?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/a43947283dd6dc9f9dc2742700c7fa4677d7e27a?src=pr&el=desc) will **not change** coverage.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/pandas-dev/pandas/pull/22038/graphs/tree.svg?token=eZ4WkYLtcO&src=pr&width=650&height=150)](https://codecov.io/gh/pandas-dev/pandas/pull/22038?src=pr&el=tree)

```diff
@@           Coverage Diff           @@
##           master   #22038   +/-   ##
=======================================
  Coverage      92%      92%           
=======================================
  Files         170      170           
  Lines       50607    50607           
=======================================
  Hits        46559    46559           
  Misses       4048     4048
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `90.4% <ø> (ø)` | :arrow_up: |
| #single | `42.19% <ø> (ø)` | :arrow_up: |


------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22038?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22038?src=pr&el=footer). Last update [a439472...43545ec](https://codecov.io/gh/pandas-dev/pandas/pull/22038?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

Is the idea here to track down the warnings that occasionally make Travis fail?
closing as fixed
Raised value error on incorrect na_option
State: closed

- [X] closes #19499
- [X] tests added / passed
- [X] passes `git diff upstream/master -u -- "*.py" | flake8 --diff`
- [ ] whatsnew entry



Labels:
name: Error Reporting
description: Incorrect or improved errors from pandas

Comments:
Also can you make sure this works for all of `Series`, `DataFrame` and `GroupBy` objects? Behavior should be identical across all three
# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22037?src=pr&el=h1) Report
> Merging [#22037](https://codecov.io/gh/pandas-dev/pandas/pull/22037?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/2d0c96119391c85bd4f7ffbb847759ee3777162a?src=pr&el=desc) will **increase** coverage by `<.01%`.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/pandas-dev/pandas/pull/22037/graphs/tree.svg?height=150&width=650&token=eZ4WkYLtcO&src=pr)](https://codecov.io/gh/pandas-dev/pandas/pull/22037?src=pr&el=tree)

```diff
@@            Coverage Diff             @@
##           master   #22037      +/-   ##
==========================================
+ Coverage      92%      92%   +<.01%     
==========================================
  Files         170      170              
  Lines       50553    50556       +3     
==========================================
+ Hits        46510    46513       +3     
  Misses       4043     4043
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `90.4% <ø> (ø)` | :arrow_up: |
| #single | `42.21% <ø> (-0.01%)` | :arrow_down: |

| [Impacted Files](https://codecov.io/gh/pandas-dev/pandas/pull/22037?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [pandas/core/generic.py](https://codecov.io/gh/pandas-dev/pandas/pull/22037/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvZ2VuZXJpYy5weQ==) | `96.47% <ø> (ø)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22037?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22037?src=pr&el=footer). Last update [2d0c961...59c98a6](https://codecov.io/gh/pandas-dev/pandas/pull/22037?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

Not sure why travis-ci is failing. Looks unrelated/might have to do with an upstream dependency? Do I have to create a new PR or can I re-run the build somehow? 
@WillAyd PTAL!
lgtm

do we have an open issue for this?
Updated original comment to make linkage clearer. Merging later today if no objections
Thanks @raguiar2!
fix Error Indexing DafaFrame with a 0-d array
State: closed

closes #21946

- [x ] closes #xxxx
- [x ] tests added / passed
- [x] passes `git diff upstream/master -u -- "*.py" | flake8 --diff`
- [x ] whatsnew entry


Labels:
name: Indexing
description: Related to indexing on series/frames, not to indexes themselves
name: Error Reporting
description: Incorrect or improved errors from pandas
name: Numeric Operations
description: Arithmetic, Comparison, and Logical operations

Comments:
Hmm..it seems after reverting my changes and taking a look at my test again that the test still passes. I think I have either a bad test or this was fixed on master before I looked at it? I'm gonna try to build from master and repro in the AM.
Yea confirmed. I checked on master today and we give out the expected error in this instance. ```TypeError: Cannot index by location index with a non-integer key``` so I'm gonna close this and comment on the other one as such. Pretty sad :'( but I'll look for something else to contribute with.
Separate out internals.concat, internals.managers
State: closed

Follow-up to #22014.

Moved `get_blockno_placements` to libinternals since a) its a natural fit anyway and b) it is needed in both concat and managers and I didn't want to runtime-import it.

The only non-cut/paste edit is a change of `isinstance(self.block, ExtensionBlock)` to `self.block.is_extension`

`__init__` namespace is big in part because `io.packers` uses `getattr(internals, name)` and is just really tightly intertwined with internals.

Labels:
name: Refactor
description: Internal refactoring of code
name: Internals
description: Related to non-user accessible pandas implementation

Comments:
# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22028?src=pr&el=h1) Report
> Merging [#22028](https://codecov.io/gh/pandas-dev/pandas/pull/22028?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/716efd3f1e08d81743edcb7422fa5eb05523b58d?src=pr&el=desc) will **increase** coverage by `<.01%`.
> The diff coverage is `96.88%`.

[![Impacted file tree graph](https://codecov.io/gh/pandas-dev/pandas/pull/22028/graphs/tree.svg?height=150&width=650&token=eZ4WkYLtcO&src=pr)](https://codecov.io/gh/pandas-dev/pandas/pull/22028?src=pr&el=tree)

```diff
@@            Coverage Diff             @@
##           master   #22028      +/-   ##
==========================================
+ Coverage   91.99%      92%   +<.01%     
==========================================
  Files         168      170       +2     
  Lines       50597    50601       +4     
==========================================
+ Hits        46549    46553       +4     
  Misses       4048     4048
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `90.4% <96.88%> (ø)` | :arrow_up: |
| #single | `42.2% <68.1%> (ø)` | :arrow_up: |

| [Impacted Files](https://codecov.io/gh/pandas-dev/pandas/pull/22028?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [pandas/core/internals/\_\_init\_\_.py](https://codecov.io/gh/pandas-dev/pandas/pull/22028/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvaW50ZXJuYWxzL19faW5pdF9fLnB5) | `100% <100%> (+3.11%)` | :arrow_up: |
| [pandas/core/internals/managers.py](https://codecov.io/gh/pandas-dev/pandas/pull/22028/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvaW50ZXJuYWxzL21hbmFnZXJzLnB5) | `96.52% <96.52%> (ø)` | |
| [pandas/core/internals/concat.py](https://codecov.io/gh/pandas-dev/pandas/pull/22028/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvaW50ZXJuYWxzL2NvbmNhdC5weQ==) | `98.37% <98.37%> (ø)` | |

------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22028?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22028?src=pr&el=footer). Last update [716efd3...f7490a9](https://codecov.io/gh/pandas-dev/pandas/pull/22028?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

yep, thanks @jbrockmendel