change default back to False

Author: reidy-p

pandas/core/frame.py

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

pandas/tests/frame/test_convert_to.py

@@ -79,19 +79,16 @@ def test_to_records_dt64(self):
                         ["four", "five", "six"]],
                        index=date_range("2012-01-01", "2012-01-02"))
 
-        with tm.assert_produces_warning(FutureWarning):
-            expected = df.index[0]
-            result = df.to_records(convert_datetime64=True)['index'][0]
-            assert expected == result
-
-        expected = df.index[0]
-        # convert_datetime64 defaults to True if not passed
+        # convert_datetime64 defaults to False if not passed
+        expected = df.index.values[0]
         result = df.to_records()['index'][0]
         assert expected == result
 
+        # check for FutureWarning if convert_datetime64=True is passed
         with tm.assert_produces_warning(FutureWarning):
-            rs = df.to_records(convert_datetime64=False)
-            assert rs['index'][0] == df.index.values[0]
+            expected = df.index[0]
+            result = df.to_records(convert_datetime64=True)['index'][0]
+            assert expected == result
 
     def test_to_records_with_multindex(self):
         # GH3189