Update examples with imports

[virajmparekh]

Variable ts1, ts2, and index are never defined in dataframe df. Should ts1 and ts2 be strings and can we get rid of the index variable?

[jorisvandenbossche]

Those two are not needed (we assume those, to not have to repeat those in each docstring. Also the tutorial docs everywhere assume those two imports)

[jorisvandenbossche]

What do you mean with "variable ts1, ts2,"?
In which docstring are they included? (I don't see them in the DataFrame one)

[TomAugspurger]

@jorisvandenbossche they are in the DataFrame class docstring

    Examples
    --------
    >>> d = {'col1': ts1, 'col2': ts2}
    >>> df = DataFrame(data=d, index=index)
    >>> df2 = DataFrame(np.random.randn(10, 5))
    >>> df3 = DataFrame(np.random.randn(10, 5),
    ...                 columns=['a', 'b', 'c', 'd', 'e'])

@vparekh94 it'd be nice to expand that a bit with text in between each explaining the data being passed in. Like

    Examples
    --------
    From a dictionary of arrays:

    >>> d = {'col1': ts1, 'col2': ts2}
    >>> df = DataFrame(data=d, index=index)

    From a 2-D numpy array:

    >>> df2 = DataFrame(np.random.randn(10, 5))

    Same, but setting the column names:

    >>> df3 = DataFrame(np.random.randn(10, 5),
    ...                 columns=['a', 'b', 'c', 'd', 'e'])

(might not have the newlines entirely correct there.

[jorisvandenbossche]

Oh, I was really blind .. :-)

But indeed, @TomAugspurger 's suggestion is the way to go!

[TomAugspurger]

@vparekh94 are you able to update this? You can push additional commits to this branch.

[codecov]

Codecov Report

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

@@            Coverage Diff             @@
##           master   #16424      +/-   ##
==========================================
- Coverage   90.42%   90.42%   -0.01%     
==========================================
  Files         161      161              
  Lines       51023    51023              
==========================================
- Hits        46138    46137       -1     
- Misses       4885     4886       +1

Flag	Coverage Δ
#multiple	88.26% <ø> (-0.01%)	⬇️
#single	40.17% <ø> (ø)	⬆️

Impacted Files	Coverage Δ
pandas/core/frame.py	97.69% <ø> (ø)	⬆️
pandas/core/common.py	91.05% <0%> (-0.34%)	⬇️

---

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 49ec31b...10463c2. Read the comment docs.

[codecov]

Codecov Report

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

@@            Coverage Diff             @@
##           master   #16424      +/-   ##
==========================================
- Coverage   90.42%   90.42%   -0.01%     
==========================================
  Files         161      161              
  Lines       51023    51023              
==========================================
- Hits        46138    46137       -1     
- Misses       4885     4886       +1

Flag	Coverage Δ
#multiple	88.26% <ø> (-0.01%)	⬇️
#single	40.17% <ø> (ø)	⬆️

Impacted Files	Coverage Δ
pandas/core/frame.py	97.69% <ø> (ø)	⬆️
pandas/core/common.py	91.05% <0%> (-0.34%)	⬇️

---

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 49ec31b...da63094. Read the comment docs.

[jreback]

can you rebase and update?

[jorisvandenbossche]

@vparekh94 sorry for the oversight on out part, but this has been fixed in the meantime in another PR #16487.
Thanks for the contribution anyway!