Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DOC: Some changes #17992

Merged
merged 1 commit into from
Oct 27, 2017
Merged

Conversation

TomAugspurger
Copy link
Contributor

  1. Added whatsnew templates for 0.21.1 and 0.22.0
  2. Update release.rst and whatsnew/v0.21.0.txt for the release tomorrow
  3. Fixed some broken references in the 0.21.0 whatsnew

@TomAugspurger TomAugspurger added this to the 0.21.0 milestone Oct 26, 2017
@TomAugspurger TomAugspurger mentioned this pull request Oct 26, 2017
64 tasks
@@ -29,13 +29,14 @@ New features
~~~~~~~~~~~~

- Support for `PEP 519 -- Adding a file system path protocol
<https://www.python.org/dev/peps/pep-0519/>`_ on most readers and writers (:issue:`13823`)
<https://www.python.org/dev/peps/pep-0519/>`_ on most readers (e.g.
:func:`pd.read_csv`) and writers (e.g. :meth:`DataFrame.to_csv`) (:issue:`13823`).
Copy link
Member

Choose a reason for hiding this comment

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

Should the "pd." be removed in :func:`pd.read_csv`?

@@ -263,16 +264,16 @@ Other Enhancements
- Integration with `Apache Parquet <https://parquet.apache.org/>`__, including a new top-level :func:`read_parquet` and :func:`DataFrame.to_parquet` method, see :ref:`here <io.parquet>`. (:issue:`15838`, :issue:`17438`)
- :func:`DataFrame.add_prefix` and :func:`DataFrame.add_suffix` now accept strings containing the '%' character. (:issue:`17151`)
- Read/write methods that infer compression (:func:`read_csv`, :func:`read_table`, :func:`read_pickle`, and :meth:`~DataFrame.to_pickle`) can now infer from path-like objects, such as ``pathlib.Path``. (:issue:`17206`)
- :func:`pd.read_sas()` now recognizes much more of the most frequently used date (datetime) formats in SAS7BDAT files. (:issue:`15871`)
- :func:`read_sas()` now recognizes much more of the most frequently used date (datetime) formats in SAS7BDAT files. (:issue:`15871`)
Copy link
Member

Choose a reason for hiding this comment

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

Should there be parenthesis in :func:`read_sas()`? Usually don't see parenthesis used with :func:, but I'm not super familiar with this, so it may still render fine.

- Added ``skipna`` parameter to :func:`~pandas.api.types.infer_dtype` to
support type inference in the presence of missing values (:issue:`17059`).
- :class:`~pandas.Resampler.nearest` is added to support nearest-neighbor upsampling (:issue:`17496`).
- :class:`~pandas.Index` has added support for a ``to_frame`` method (:issue:`15230`)
- :class:`~pandas.core.resample.Resampler.nearest` is added to support nearest-neighbor upsampling (:issue:`17496`).
Copy link
Member

Choose a reason for hiding this comment

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

Should this be :meth: instead of :class:?

@TomAugspurger
Copy link
Contributor Author

Thanks @jschendel, you're right on all counts I think.

@codecov
Copy link

codecov bot commented Oct 26, 2017

Codecov Report

Merging #17992 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #17992      +/-   ##
==========================================
- Coverage   91.23%   91.22%   -0.02%     
==========================================
  Files         163      163              
  Lines       50113    50113              
==========================================
- Hits        45723    45714       -9     
- Misses       4390     4399       +9
Flag Coverage Δ
#multiple 89.03% <ø> (ø) ⬆️
#single 40.31% <ø> (-0.06%) ⬇️
Impacted Files Coverage Δ
pandas/io/gbq.py 25% <0%> (-58.34%) ⬇️
pandas/core/frame.py 97.75% <0%> (-0.1%) ⬇️

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 6779ac0...611c6d5. Read the comment docs.

@codecov
Copy link

codecov bot commented Oct 26, 2017

Codecov Report

Merging #17992 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #17992      +/-   ##
==========================================
- Coverage   91.26%   91.22%   -0.05%     
==========================================
  Files         163      163              
  Lines       50117    50117              
==========================================
- Hits        45739    45718      -21     
- Misses       4378     4399      +21
Flag Coverage Δ
#multiple 89.03% <ø> (-0.03%) ⬇️
#single 40.3% <ø> (-0.06%) ⬇️
Impacted Files Coverage Δ
pandas/io/gbq.py 25% <0%> (-58.34%) ⬇️
pandas/plotting/_converter.py 63.38% <0%> (-1.82%) ⬇️
pandas/core/frame.py 97.75% <0%> (-0.1%) ⬇️

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 7aeccd3...ae74a92. Read the comment docs.

@@ -95,6 +97,7 @@ HDFStore: PyTables (HDF5)
:toctree: generated/

read_hdf
HDFStore
Copy link
Member

Choose a reason for hiding this comment

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

This will also document all methods/attributes (and not all of them are documented in the user guide I think). Not familiar with this part of the code, but just wondering to what extent we need to publicize them (by having html docstring pages for them and listed on the HDFStore page)
(could also use the class_without_autosummary)

Copy link
Contributor Author

@TomAugspurger TomAugspurger Oct 27, 2017

Choose a reason for hiding this comment

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

Oh right, I forgot we included everything. The only reason I included it was since I referenced it about the __fspath__ stuff, but that alone isn't a good enough reason. I'll just remove them and the references for now. I really just want the class docstring, but we can do that later.

.. _whatsnew_0211.api_breaking:

Backwards incompatible API changes
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Copy link
Member

Choose a reason for hiding this comment

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

There should be no breaking changes in 0.21.1
(I know there can be if needed, but we should not encourage ourselves :-))

.. _whatsnew_0211.bug_fixes:

Bug Fixes
~~~~~~~~~
Copy link
Contributor

Choose a reason for hiding this comment

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

can you add some blank lines in each of the bug fix sub-sectios (and some of the other sections as well). this helps prevent conflicts (if people don't all pick the same line).

1. Added whatsnew templates for 0.21.1 and 0.22.0
2. Update release.rst and whatsnew/v0.21.0.txt for the release tomorrow
3. Fixed some broken references in the 0.21.0 whatsnew
@jorisvandenbossche jorisvandenbossche merged commit 79498e3 into pandas-dev:master Oct 27, 2017
@TomAugspurger TomAugspurger deleted the doc-templates branch October 27, 2017 12:04
peterpanmj pushed a commit to peterpanmj/pandas that referenced this pull request Oct 31, 2017
1. Added whatsnew templates for 0.21.1 and 0.22.0
2. Update release.rst and whatsnew/v0.21.0.txt for the release tomorrow
3. Fixed some broken references in the 0.21.0 whatsnew
No-Stream pushed a commit to No-Stream/pandas that referenced this pull request Nov 28, 2017
1. Added whatsnew templates for 0.21.1 and 0.22.0
2. Update release.rst and whatsnew/v0.21.0.txt for the release tomorrow
3. Fixed some broken references in the 0.21.0 whatsnew
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants