changing default to None

Author: reidy-p

doc/source/whatsnew/v0.23.0.txt

@@ -632,7 +632,7 @@ Deprecations
 - The ``broadcast`` parameter of ``.apply()`` is deprecated in favor of ``result_type='broadcast'`` (:issue:`18577`)
 - The ``reduce`` parameter of ``.apply()`` is deprecated in favor of ``result_type='reduce'`` (:issue:`18577`)
 - The ``order`` parameter of :func:`factorize` is deprecated and will be removed in a future release (:issue:`19727`)
-- The ``convert_datetime64`` parameter in :func:`DataFrame.to_records` has been deprecated and the default value is now ``False``. The NumPy bug motivating this parameter has been resolved (:issue:`18160`).
+- The ``convert_datetime64`` parameter in :func:`DataFrame.to_records` has been deprecated and will be removed in a future version. The NumPy bug motivating this parameter has been resolved (:issue:`18160`).
 
 .. _whatsnew_0230.prior_deprecations:
 

pandas/core/frame.py

@@ -1203,7 +1203,7 @@ def from_records(cls, data, index=None, exclude=None, columns=None,
 
         return cls(mgr)
 
-    def to_records(self, index=True, convert_datetime64=False):
+    def to_records(self, index=True, convert_datetime64=None):
         """
         Convert DataFrame to record array. Index will be put in the
         'index' field of the record array if requested
@@ -1212,7 +1212,7 @@ def to_records(self, index=True, convert_datetime64=False):
         ----------
         index : boolean, default True
             Include index in resulting record array, stored in 'index' field
-        convert_datetime64 : boolean, default False
+        convert_datetime64 : boolean, optional
             .. deprecated:: 0.23.0
 
             Whether to convert the index to datetime.datetime if it is a
@@ -1223,7 +1223,7 @@ def to_records(self, index=True, convert_datetime64=False):
         y : recarray
         """
 
-        if convert_datetime64:
+        if convert_datetime64 is not None:
             warnings.warn("The 'convert_datetime64' parameter is "
                           "deprecated and will be removed in a future "
                           "version",

pandas/tests/frame/test_convert_to.py

@@ -83,8 +83,9 @@ def test_to_records_dt64(self):
             result = df.to_records(convert_datetime64=True)['index'][0]
             assert expected == result
 
-        rs = df.to_records(convert_datetime64=False)
-        assert rs['index'][0] == df.index.values[0]
+        with tm.assert_produces_warning(FutureWarning):
+            rs = df.to_records(convert_datetime64=False)
+            assert rs['index'][0] == df.index.values[0]
 
     def test_to_records_with_multindex(self):
         # GH3189