order, extra docs

crates/polars-time/src/utils.rs

@@ -15,9 +15,11 @@ use polars_core::prelude::PolarsResult;
 ///
 /// e.g. '2021-01-01 03:00' -> '2021-01-01 03:00CDT'
 ///
-/// Note: this may only return `Ok(None)` if ambiguous is Ambiguous::Null.
+/// Note: this may only return `Ok(None)` if ambiguous is Ambiguous::Null
+/// or if non_existent is NonExistent::Null.
 /// Otherwise, it will either return `Ok(Some(NaiveDateTime))` or `PolarsError`.
-/// Therefore, calling `try_localize_datetime(..., Ambiguous::Raise)?.unwrap()`
+///
+/// Therefore, calling `try_localize_datetime(..., Ambiguous::Raise, NonExistent::Raise)?.unwrap()`
 /// is safe, and will never panic.
 #[cfg(feature = "timezones")]
 pub(crate) fn try_localize_datetime(

py-polars/src/conversion/mod.rs

@@ -712,21 +712,6 @@ impl FromPyObject<'_> for Wrap<ClosedWindow> {
     }
 }
 
-impl FromPyObject<'_> for Wrap<NonExistent> {
-    fn extract(ob: &PyAny) -> PyResult<Self> {
-        let parsed = match ob.extract::<&str>()? {
-            "null" => NonExistent::Null,
-            "raise" => NonExistent::Raise,
-            v => {
-                return Err(PyValueError::new_err(format!(
-                    "`non_existent` must be one of {{'null', 'raise'}}, got {v}",
-                )))
-            },
-        };
-        Ok(Wrap(parsed))
-    }
-}
-
 #[cfg(feature = "csv")]
 impl FromPyObject<'_> for Wrap<CsvEncoding> {
     fn extract(ob: &PyAny) -> PyResult<Self> {
@@ -812,6 +797,21 @@ impl FromPyObject<'_> for Wrap<ListToStructWidthStrategy> {
     }
 }
 
+impl FromPyObject<'_> for Wrap<NonExistent> {
+    fn extract(ob: &PyAny) -> PyResult<Self> {
+        let parsed = match ob.extract::<&str>()? {
+            "null" => NonExistent::Null,
+            "raise" => NonExistent::Raise,
+            v => {
+                return Err(PyValueError::new_err(format!(
+                    "`non_existent` must be one of {{'null', 'raise'}}, got {v}",
+                )))
+            },
+        };
+        Ok(Wrap(parsed))
+    }
+}
+
 impl FromPyObject<'_> for Wrap<NullBehavior> {
     fn extract(ob: &PyAny) -> PyResult<Self> {
         let parsed = match ob.extract::<&str>()? {