CLN: removed unused "convert" parameter to _take

[toobaz]

• passes git diff upstream/master -u -- "*.py" | flake8 --diff

Followup to #20841: the convert parameter to _take is irrelevant: the conversion of negative to positive indices is required only for Categoricals (and only until we normalize their take behavior with negative indices).

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (master@2ab3727). Click here to learn what that means.
The diff coverage is 84%.

@@            Coverage Diff            @@
##             master   #20934   +/-   ##
=========================================
  Coverage          ?   91.81%           
=========================================
  Files             ?      153           
  Lines             ?    49474           
  Branches          ?        0           
=========================================
  Hits              ?    45424           
  Misses            ?     4050           
  Partials          ?        0

Flag	Coverage Δ
#multiple	90.21% <84%> (?)
#single	41.85% <36%> (?)

Impacted Files	Coverage Δ
pandas/core/groupby/groupby.py	92.58% <100%> (ø)
pandas/core/series.py	94.02% <100%> (ø)
pandas/core/indexing.py	93.55% <71.42%> (ø)
pandas/core/frame.py	97.13% <75%> (ø)
pandas/core/generic.py	95.93% <87.5%> (ø)

---

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 2ab3727...7aa63c2. Read the comment docs.

[jreback]

thanks!

FYI a more complete linter command is:

git diff master --name-only -- "*.py" | grep "pandas" | xargs flake8

(which catches unused things as well)