DOC: avoid SparseArray.take error
State: closed

Closes https://github.com/pandas-dev/pandas/issues/22215

SparseArray.take not accepting scalars is already in 0.24.0.txt

Labels:
name: Docs
name: Sparse
description: Sparse Data Type

Comments:
This is "fixing" the doc build warning by not executing it. We changed API
to not allow scalars in SparseArray.take

On Mon, Nov 12, 2018 at 8:40 AM Jeff Reback <notifications@github.com>
wrote:

> *@jreback* commented on this pull request.
> ------------------------------
>
> In doc/source/whatsnew/v0.18.1.txt
> <https://github.com/pandas-dev/pandas/pull/23637#discussion_r232681720>:
>
> > @@ -266,7 +266,7 @@ These changes conform sparse handling to return the correct types and work to ma
>
>  ``SparseArray.take`` now returns a scalar for scalar input, ``SparseArray`` for others. Furthermore, it handles a negative indexer with the same rule as ``Index`` (:issue:`10560`, :issue:`12796`)
>
> -.. ipython:: python
> +.. code-block:: python
>
> is this supposed to work?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/pandas-dev/pandas/pull/23637#pullrequestreview-173930038>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ABQHIsGaiZOcxjXbZcXJmKUKWWs0lXSyks5uuYhNgaJpZM4YZpe1>
> .
>

# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/23637?src=pr&el=h1) Report
> Merging [#23637](https://codecov.io/gh/pandas-dev/pandas/pull/23637?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/011b79fbf73b45313b47c08b4be1fc07dcb99365?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/23637/graphs/tree.svg?width=650&token=eZ4WkYLtcO&height=150&src=pr)](https://codecov.io/gh/pandas-dev/pandas/pull/23637?src=pr&el=tree)

```diff
@@           Coverage Diff           @@
##           master   #23637   +/-   ##
=======================================
  Coverage   92.24%   92.24%           
=======================================
  Files         161      161           
  Lines       51326    51326           
=======================================
  Hits        47347    47347           
  Misses       3979     3979
```

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


------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/23637?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/23637?src=pr&el=footer). Last update [011b79f...35bd824](https://codecov.io/gh/pandas-dev/pandas/pull/23637?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

> We changed API to not allow scalars in SparseArray.take

wait, so then that should just raise?


It does raise on master. This is in the 0.18.1 release notes.

On Mon, Nov 12, 2018 at 9:39 AM Jeff Reback <notifications@github.com>
wrote:

> We changed API to not allow scalars in SparseArray.take
>
> wait, so then that should just raise?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/pandas-dev/pandas/pull/23637#issuecomment-437926273>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ABQHIuccwZ4AgIKJyh4LezXL_Y_Thr1kks5uuZYpgaJpZM4YZpe1>
> .
>

oh! ic. ok then.