deprecated get_original_spans

python/src/lib.rs

@@ -1,6 +1,6 @@
 #![allow(clippy::deprecated)]
 use pyo3::prelude::*;
-use tokenizations::{get_alignments, get_charmap, get_original_spans, Alignment, CharMap};
+use tokenizations::{get_alignments, get_charmap, Alignment, CharMap};
 
 #[pymodule]
 fn tokenizations(_py: Python, m: &PyModule) -> PyResult<()> {
@@ -20,14 +20,5 @@ fn tokenizations(_py: Python, m: &PyModule) -> PyResult<()> {
         Ok(get_charmap(a, b))
     }
 
-    #[pyfn(m, "get_original_spans")]
-    pub fn get_original_spans_py(
-        _py: Python,
-        tokens: Vec<&str>,
-        original_text: &str,
-    ) -> PyResult<Vec<Option<(usize, usize)>>> {
-        Ok(get_original_spans(&tokens, original_text))
-    }
-
     Ok(())
 }

python/tests/test_main.py

@@ -75,6 +75,5 @@ def test_warn_get_original_spans():
     reason="deprecation error check",
 )
 def test_error_get_original_spans():
-    with pytest.raises(Exception):
+    with pytest.raises(ValueError):
         get_original_spans([], "")
-

python/tokenizations/__init__.py

@@ -2,17 +2,14 @@
 from .tokenizations import (
     get_alignments,
     get_charmap,
-    get_original_spans as _get_original_spans,
     __version__,
 )
 
 
 def get_original_spans(tokens, original_text):
-    warnings.warn(
-        "get_original_spans is deprecated, use `textspan.get_original_spans` instead.",
-        DeprecationWarning,
+    raise ValueError(
+        f"{get_original_spans.__name__} was deprecated. Please use `textspan.get_original_spans` instead."
     )
-    return _get_original_spans(tokens, original_text)
 
 
 __all__ = ["get_charmap", "get_alignments", "get_original_spans", "__version__"]