python-calamine 0.4.0__tar.gz → 0.5.1__tar.gz

{python_calamine-0.4.0 → python_calamine-0.5.1}/.github/workflows/CI.yml

@@ -13,33 +13,29 @@ on:
 
 jobs:
   test:
-    name: test ${{ matrix.python-version }} pandas ${{ matrix.pandas-version }} ${{ matrix.disable-gil && 'free-threaded' || '' }}
+    name: test ${{ matrix.python-version }} pandas ${{ matrix.pandas-version }}
     strategy:
       fail-fast: false
       matrix:
         python-version:
-          - "3.9"
           - "3.10"
           - "3.11"
           - "3.12"
+          - "3.13"
+          - "3.14"
         pandas-version:
-          - "2.1.*"
           - "2.2.*"
+          - "2.3.*"
         include:
-          - python-version: "3.8"
-            pandas-version: "none"
           # https://github.com/pandas-dev/pandas/issues/42509
-          - python-version: "pypy3.9"
+          - python-version: "pypy3.11"
             pandas-version: "none"
-          # https://github.com/pandas-dev/pandas/issues/42509
-          - python-version: "pypy3.10"
+          # https://github.com/pandas-dev/pandas/issues/60016
+          - python-version: "3.13t"
             pandas-version: "none"
-          - python-version: "3.13"
-            pandas-version: "2.2.*"
           # https://github.com/pandas-dev/pandas/issues/60016
-          - python-version: "3.13"
+          - python-version: "3.14t"
             pandas-version: "none"
-            disable-gil: true
 
     runs-on: ubuntu-latest
 
@@ -47,7 +43,7 @@ jobs:
       PYTHON: ${{ matrix.python-version }}
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v5
 
       - name: install rust stable
         uses: dtolnay/rust-toolchain@master
@@ -61,29 +57,22 @@ jobs:
           key: v3
 
       - name: set up python
-        uses: actions/setup-python@v5
+        uses: actions/setup-python@v6
         if: "${{ !matrix.disable-gil }}"
         with:
           python-version: ${{ matrix.python-version }}
           allow-prereleases: true
 
-      - name: set up python (free-threaded)
-        uses: deadsnakes/action@v3.2.0
-        if: "${{ matrix.disable-gil }}"
-        with:
-          python-version: ${{ matrix.python-version }}
-          nogil: ${{ matrix.disable-gil }}
-
       - id: cache-py
         name: cache python
         uses: actions/cache@v4
-        if: "${{ !matrix.disable-gil }}"
         with:
           path: ${{ env.pythonLocation }}
           key: >
             py
             ${{ runner.os }}
-            ${{ env.pythonLocation }}
+            ${{ matrix.python-version }}
+            ${{ matrix.pandas-version }}
 
       - run: |
           if [ ${{ matrix.pandas-version }} == 'none' ]; then
@@ -105,7 +94,7 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v5
 
       - name: install rust stable
         uses: dtolnay/rust-toolchain@stable
@@ -115,7 +104,7 @@ jobs:
       - name: cache rust
         uses: Swatinem/rust-cache@v2
 
-      - uses: actions/setup-python@v5
+      - uses: actions/setup-python@v6
         with:
           python-version: "3.13"
 
@@ -155,7 +144,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os: [ubuntu, macos, windows]
+        os: [ubuntu, macos]
         target: [x86_64, aarch64]
         manylinux: [auto]
         include:
@@ -163,15 +152,17 @@ jobs:
             platform: linux
           - os: windows
             ls: dir
+            target: x86_64
+            interpreter: 3.10 3.11 3.12 3.13 3.13t
           - os: windows
             ls: dir
             target: i686
             python-architecture: x86
-            interpreter: 3.8 3.9 3.10 3.11 3.12 3.13
+            interpreter: 3.10 3.11 3.12 3.13 3.13t
           - os: windows
             ls: dir
             target: aarch64
-            interpreter: 3.11 3.12 3.13
+            interpreter: 3.11 3.12 3.13 3.13t
           - os: ubuntu
             platform: linux
             target: i686
@@ -181,7 +172,7 @@ jobs:
           - os: ubuntu
             platform: linux
             target: armv7
-            interpreter: 3.8 3.9 3.10 3.11 3.12 3.13
+            interpreter: 3.10 3.11 3.12 3.13 3.13t 3.14 3.14t pypy3.11
             # musllinux
           - os: ubuntu
             platform: linux
@@ -194,22 +185,18 @@ jobs:
           - os: ubuntu
             platform: linux
             target: ppc64le
-            interpreter: 3.8 3.9 3.10 3.11 3.12 3.13
+            interpreter: 3.10 3.11 3.12 3.13 3.13t 3.14 3.14t pypy3.11
           - os: ubuntu
             platform: linux
             target: s390x
-            interpreter: 3.8 3.9 3.10 3.11 3.12 3.13
-        exclude:
-          # Windows on arm64 only supports Python 3.11+
-          - os: windows
-            target: aarch64
+            interpreter: 3.10 3.11 3.12 3.13 3.13t 3.14 3.14t pypy3.11
 
     runs-on: ${{ matrix.os }}-latest
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v5
 
       - name: set up python
-        uses: actions/setup-python@v5
+        uses: actions/setup-python@v6
         with:
           python-version: "3.13"
           architecture: ${{ matrix.python-architecture || 'x64' }}
@@ -229,7 +216,7 @@ jobs:
         with:
           target: ${{ matrix.target }}
           manylinux: ${{ matrix.manylinux || 'auto' }}
-          args: --release --out dist --interpreter ${{ matrix.interpreter || '3.8 3.9 3.10 3.11 3.12 3.13 pypy3.9 pypy3.10' }}
+          args: --release --out dist --interpreter ${{ matrix.interpreter || '3.10 3.11 3.12 3.13 3.13t 3.14 3.14t pypy3.11' }}
           rust-toolchain: stable
 
       - run: ${{ matrix.ls || 'ls -lh' }} dist/
@@ -256,7 +243,7 @@ jobs:
 
     steps:
       - name: Download All Artifacts
-        uses: actions/download-artifact@v4
+        uses: actions/download-artifact@v5
         with:
           path: dist
           pattern: wheels-*

{python_calamine-0.4.0 → python_calamine-0.5.1}/.pre-commit-config.yaml

@@ -38,8 +38,6 @@ repos:
       rev: v1.8.0
       hooks:
         - id: mypy
-          additional_dependencies:
-            - pandas-stubs
           exclude: ^tests/.*$
     -   repo: local
         hooks:

{python_calamine-0.4.0 → python_calamine-0.5.1}/Cargo.lock

@@ -70,8 +70,9 @@ checksum = "1fd0f2584146f6f2ef48085050886acf353beff7305ebd1ae69500e27c67f64b"
 
 [[package]]
 name = "calamine"
-version = "0.26.1"
-source = "git+https://github.com/tafia/calamine.git?rev=5cda5c734661f656bdfb7ac7a1b95e77bf89aa09#5cda5c734661f656bdfb7ac7a1b95e77bf89aa09"
+version = "0.30.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "a9aeb09f84576a634da713630e11e431a744b91f1f8114c2ff0760189783a8a1"
 dependencies = [
  "atoi_simd",
  "byteorder",
@@ -170,12 +171,6 @@ dependencies = [
  "cfg-if",
 ]
 
-[[package]]
-name = "crossbeam-utils"
-version = "0.8.21"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "d0a5c400df2834b80a4c3327b3aad3a4c4cd4de0629063962b03235697506a28"
-
 [[package]]
 name = "derive_arbitrary"
 version = "1.4.1"
@@ -187,17 +182,6 @@ dependencies = [
  "syn",
 ]
 
-[[package]]
-name = "displaydoc"
-version = "0.2.5"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "97369cbbc041bc366949bc74d34658d6cda5621039731c6310521892a3a20ae0"
-dependencies = [
- "proc-macro2",
- "quote",
- "syn",
-]
-
 [[package]]
 name = "encoding_rs"
 version = "0.8.35"
@@ -246,11 +230,12 @@ checksum = "37909eebbb50d72f9059c3b6d82c0463f2ff062c9e95845c43a6c9c0355411be"
 
 [[package]]
 name = "flate2"
-version = "1.0.35"
+version = "1.1.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "c936bfdafb507ebbf50b8074c54fa31c5be9a1e7e5f467dd659697041407d07c"
+checksum = "4a3d7db9596fecd151c5f638c0ee5d5bd487b6e0ea232e5dc96d5250f6f94b1d"
 dependencies = [
  "crc32fast",
+ "libz-rs-sys",
  "miniz_oxide",
 ]
 
@@ -344,6 +329,15 @@ version = "0.2.169"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "b5aba8db14291edd000dfcc4d620c7ebfb122c613afb886ca8803fa4e128a20a"
 
+[[package]]
+name = "libz-rs-sys"
+version = "0.5.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "172a788537a2221661b480fee8dc5f96c580eb34fa88764d3205dc356c7e4221"
+dependencies = [
+ "zlib-rs",
+]
+
 [[package]]
 name = "linux-raw-sys"
 version = "0.4.15"
@@ -379,9 +373,9 @@ dependencies = [
 
 [[package]]
 name = "miniz_oxide"
-version = "0.8.2"
+version = "0.8.9"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "4ffbe83022cedc1d264172192511ae958937694cd57ce297164951b8b3568394"
+checksum = "1fa76a2c86f704bdb222d66965fb3d63269ce38518b83cb0575fca855ebb6316"
 dependencies = [
  "adler2",
 ]
@@ -397,9 +391,9 @@ dependencies = [
 
 [[package]]
 name = "once_cell"
-version = "1.20.2"
+version = "1.21.3"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "1261fe7e33c73b354eab43b1273a57c8f967d0391e80353e51f764ac02cf6775"
+checksum = "42f5e15c9953c5e4ccceeb2e7382a716482c34515315f7b03532b8b4e8393d2d"
 
 [[package]]
 name = "portable-atomic"
@@ -429,9 +423,9 @@ dependencies = [
 
 [[package]]
 name = "pyo3"
-version = "0.25.1"
+version = "0.26.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "8970a78afe0628a3e3430376fc5fd76b6b45c4d43360ffd6cdd40bdde72b682a"
+checksum = "7ba0117f4212101ee6544044dae45abe1083d30ce7b29c4b5cbdfa2354e07383"
 dependencies = [
  "chrono",
  "indoc",
@@ -447,20 +441,19 @@ dependencies = [
 
 [[package]]
 name = "pyo3-build-config"
-version = "0.25.1"
+version = "0.26.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "458eb0c55e7ece017adeba38f2248ff3ac615e53660d7c71a238d7d2a01c7598"
+checksum = "4fc6ddaf24947d12a9aa31ac65431fb1b851b8f4365426e182901eabfb87df5f"
 dependencies = [
- "once_cell",
  "python3-dll-a",
  "target-lexicon",
 ]
 
 [[package]]
 name = "pyo3-ffi"
-version = "0.25.1"
+version = "0.26.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "7114fe5457c61b276ab77c5055f206295b812608083644a5c5b2640c3102565c"
+checksum = "025474d3928738efb38ac36d4744a74a400c901c7596199e20e45d98eb194105"
 dependencies = [
  "libc",
  "pyo3-build-config",
@@ -469,8 +462,7 @@ dependencies = [
 [[package]]
 name = "pyo3-file"
 version = "0.13.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "75916300df50dbfa6958ec05fe53473e69474542dac7fd1275aa6869a18a241a"
+source = "git+https://github.com/dimastbk/pyo3-file.git?rev=da8ddb5d774d042555b867dd6571f3ce128bc349#da8ddb5d774d042555b867dd6571f3ce128bc349"
 dependencies = [
  "pyo3",
  "skeptic",
@@ -478,9 +470,9 @@ dependencies = [
 
 [[package]]
 name = "pyo3-macros"
-version = "0.25.1"
+version = "0.26.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "a8725c0a622b374d6cb051d11a0983786448f7785336139c3c94f5aa6bef7e50"
+checksum = "2e64eb489f22fe1c95911b77c44cc41e7c19f3082fc81cce90f657cdc42ffded"
 dependencies = [
  "proc-macro2",
  "pyo3-macros-backend",
@@ -490,9 +482,9 @@ dependencies = [
 
 [[package]]
 name = "pyo3-macros-backend"
-version = "0.25.1"
+version = "0.26.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "4109984c22491085343c05b0dbc54ddc405c3cf7b4374fc533f5c3313a572ccc"
+checksum = "100246c0ecf400b475341b8455a9213344569af29a3c841d29270e53102e0fcf"
 dependencies = [
  "heck",
  "proc-macro2",
@@ -503,7 +495,7 @@ dependencies = [
 
 [[package]]
 name = "python-calamine"
-version = "0.4.0"
+version = "0.5.1"
 dependencies = [
  "calamine",
  "chrono",
@@ -667,26 +659,6 @@ dependencies = [
  "windows-sys",
 ]
 
-[[package]]
-name = "thiserror"
-version = "2.0.10"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "a3ac7f54ca534db81081ef1c1e7f6ea8a3ef428d2fc069097c079443d24124d3"
-dependencies = [
- "thiserror-impl",
-]
-
-[[package]]
-name = "thiserror-impl"
-version = "2.0.10"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "9e9465d30713b56a37ede7185763c3492a91be2f5fa68d958c44e41ab9248beb"
-dependencies = [
- "proc-macro2",
- "quote",
- "syn",
-]
-
 [[package]]
 name = "unicase"
 version = "2.8.1"
@@ -880,21 +852,24 @@ checksum = "589f6da84c646204747d1270a2a5661ea66ed1cced2631d546fdfb155959f9ec"
 
 [[package]]
 name = "zip"
-version = "2.4.1"
+version = "4.5.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "938cc23ac49778ac8340e366ddc422b2227ea176edb447e23fc0627608dddadd"
+checksum = "8835eb39822904d39cb19465de1159e05d371973f0c6df3a365ad50565ddc8b9"
 dependencies = [
  "arbitrary",
  "crc32fast",
- "crossbeam-utils",
- "displaydoc",
  "flate2",
  "indexmap",
  "memchr",
- "thiserror",
  "zopfli",
 ]
 
+[[package]]
+name = "zlib-rs"
+version = "0.5.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "626bd9fa9734751fc50d6060752170984d7053f5a39061f524cda68023d4db8a"
+
 [[package]]
 name = "zopfli"
 version = "0.8.1"

python_calamine-0.5.1/Cargo.toml

@@ -0,0 +1,22 @@
+[package]
+name = "python-calamine"
+version = "0.5.1"
+edition = "2021"
+readme = "README.md"
+
+[lib]
+name = "python_calamine"
+crate-type = ["cdylib"]
+
+[dependencies]
+calamine = { version = "0.30.0", features = ["dates"] }
+pyo3 = { version = "0.26.0", features = [
+    "extension-module",
+    "chrono",
+    "generate-import-lib",
+] }
+chrono = { version = "0.4.38", features = ["serde"] }
+pyo3-file = { git = "https://github.com/dimastbk/pyo3-file.git", rev = "da8ddb5d774d042555b867dd6571f3ce128bc349" }
+
+[build-dependencies]
+pyo3-build-config = "0.26.0"

{python_calamine-0.4.0 → python_calamine-0.5.1}/PKG-INFO

@@ -1,31 +1,24 @@
 Metadata-Version: 2.4
 Name: python-calamine
-Version: 0.4.0
+Version: 0.5.1
 Classifier: Development Status :: 4 - Beta
-Classifier: License :: OSI Approved :: MIT License
 Classifier: Programming Language :: Rust
-Classifier: Programming Language :: Python :: 3.8
-Classifier: Programming Language :: Python :: 3.9
 Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.13
-Requires-Dist: packaging>=23.1
+Classifier: Programming Language :: Python :: 3.14
 Requires-Dist: maturin~=1.0 ; extra == 'dev'
-Requires-Dist: pre-commit~=3.0 ; extra == 'dev'
+Requires-Dist: pre-commit~=4.3 ; extra == 'dev'
 Requires-Dist: pytest~=8.0 ; extra == 'dev'
-Requires-Dist: pandas[excel]~=2.0 ; extra == 'dev'
-Requires-Dist: numpy~=1.0 ; extra == 'dev'
+Requires-Dist: pandas[excel]~=2.2 ; extra == 'dev'
 Provides-Extra: dev
 License-File: LICENSE
 Summary: Python binding for Rust's library for reading excel and odf file - calamine
-Home-Page: https://github.com/dimastbk/python-calamine
-Author: Dmitriy <dimastbk@proton.me>
 Author-email: Dmitriy <dimastbk@proton.me>
-License: MIT
-Requires-Python: >=3.8
+License-Expression: MIT
+Requires-Python: >=3.10
 Description-Content-Type: text/markdown; charset=UTF-8; variant=GFM
-Project-URL: Source Code, https://github.com/dimastbk/python-calamine
 
 # python-calamine
 [![PyPI - Version](https://img.shields.io/pypi/v/python-calamine)](https://pypi.org/project/python-calamine/)
@@ -79,20 +72,7 @@ workbook = CalamineWorkbook.from_path("file.xlsx").get_sheet_by_name("Sheet1").t
 # ]
 ```
 
-Also, you can use monkeypatch for pandas for use this library as engine in `read_excel()` (only pandas 2.0 and 2.1 are supported).
 Pandas 2.2 and above have built-in support of python-calamine.
-```python
-from pandas import read_excel
-from python_calamine.pandas import pandas_monkeypatch
-
-
-pandas_monkeypatch()
-
-read_excel("file.xlsx", engine="calamine")
-#            1   2   3   4   5   6   7
-# 0          1   2   3   4   5   6   7
-# 1          1   2   3   4   5   6   7
-```
 
 Also, you can find additional examples in [tests](https://github.com/dimastbk/python-calamine/blob/master/tests/test_base.py).
 

{python_calamine-0.4.0 → python_calamine-0.5.1}/README.md

@@ -50,20 +50,7 @@ workbook = CalamineWorkbook.from_path("file.xlsx").get_sheet_by_name("Sheet1").t
 # ]
 ```
 
-Also, you can use monkeypatch for pandas for use this library as engine in `read_excel()` (only pandas 2.0 and 2.1 are supported).
 Pandas 2.2 and above have built-in support of python-calamine.
-```python
-from pandas import read_excel
-from python_calamine.pandas import pandas_monkeypatch
-
-
-pandas_monkeypatch()
-
-read_excel("file.xlsx", engine="calamine")
-#            1   2   3   4   5   6   7
-# 0          1   2   3   4   5   6   7
-# 1          1   2   3   4   5   6   7
-```
 
 Also, you can find additional examples in [tests](https://github.com/dimastbk/python-calamine/blob/master/tests/test_base.py).
 

{python_calamine-0.4.0 → python_calamine-0.5.1}/pyproject.toml

@@ -1,27 +1,27 @@
 [project]
 name = "python-calamine"
-requires-python = ">=3.8"
+description = "Python binding for Rust's library for reading excel and odf file - calamine"
+readme = "README.md"
+requires-python = ">=3.10"
+license = "MIT"
+authors = [{ name = "Dmitriy", email = "dimastbk@proton.me" }]
 classifiers = [
   "Development Status :: 4 - Beta",
-  "License :: OSI Approved :: MIT License",
   "Programming Language :: Rust",
-  "Programming Language :: Python :: 3.8",
-  "Programming Language :: Python :: 3.9",
   "Programming Language :: Python :: 3.10",
   "Programming Language :: Python :: 3.11",
   "Programming Language :: Python :: 3.12",
   "Programming Language :: Python :: 3.13",
+  "Programming Language :: Python :: 3.14",
 ]
-dependencies = ["packaging>=23.1"]
 dynamic = ["version"]
 
 [project.optional-dependencies]
 dev = [
   "maturin~=1.0",
-  "pre-commit~=3.0",
+  "pre-commit~=4.3",
   "pytest~=8.0",
-  "pandas[excel]~=2.0",
-  "numpy~=1.0",
+  "pandas[excel]~=2.2",
 ]
 
 [tool.isort]
@@ -31,7 +31,7 @@ multi_line_output = 3
 profile = "black"
 
 [tool.mypy]
-python_version = "3.8"
+python_version = "3.10"
 ignore_missing_imports = false
 disallow_untyped_defs = true
 check_untyped_defs = true

{python_calamine-0.4.0 → python_calamine-0.5.1}/src/lib.rs

@@ -8,7 +8,7 @@ use crate::types::{
 };
 
 #[pyfunction]
-fn load_workbook(py: Python, path_or_filelike: PyObject) -> PyResult<CalamineWorkbook> {
+fn load_workbook(py: Python, path_or_filelike: Py<PyAny>) -> PyResult<CalamineWorkbook> {
     CalamineWorkbook::from_object(py, path_or_filelike)
 }
 

{python_calamine-0.4.0 → python_calamine-0.5.1}/src/types/cell.rs

@@ -1,8 +1,16 @@
 use std::convert::From;
 
 use calamine::DataType;
+use chrono::Datelike;
 use pyo3::prelude::*;
 
+/// https://docs.python.org/3/library/datetime.html#constants
+/// The smallest year number allowed in a date or datetime object. MINYEAR is 1.
+const MINYEAR: i32 = 1;
+
+/// The largest year number allowed in a date or datetime object. MAXYEAR is 9999.
+const MAXYEAR: i32 = 9999;
+
 #[derive(Debug, Clone)]
 pub enum CellValue {
     Int(i64),
@@ -16,6 +24,14 @@ pub enum CellValue {
     Empty,
 }
 
+fn check_year_range<DT: Datelike>(value: DT) -> Option<DT> {
+    if value.year() < MINYEAR || value.year() > MAXYEAR {
+        None
+    } else {
+        Some(value)
+    }
+}
+
 impl<'py> IntoPyObject<'py> for CellValue {
     type Target = PyAny;
     type Output = Bound<'py, Self::Target>;
@@ -64,19 +80,31 @@ where
             } else if v < 1.0 {
                 value.as_time().map(CellValue::Time)
             } else if v == (v as u64) as f64 {
-                value.as_date().map(CellValue::Date)
+                value
+                    .as_date()
+                    .and_then(check_year_range)
+                    .map(CellValue::Date)
             } else {
-                value.as_datetime().map(CellValue::DateTime)
+                value
+                    .as_datetime()
+                    .and_then(check_year_range)
+                    .map(CellValue::DateTime)
             }
             .unwrap_or(CellValue::Float(v))
         } else if value.is_datetime_iso() {
             let v = value.get_datetime_iso().unwrap();
             if v.contains('T') {
-                value.as_datetime().map(CellValue::DateTime)
+                value
+                    .as_datetime()
+                    .and_then(check_year_range)
+                    .map(CellValue::DateTime)
             } else if v.contains(':') {
                 value.as_time().map(CellValue::Time)
             } else {
-                value.as_date().map(CellValue::Date)
+                value
+                    .as_date()
+                    .and_then(check_year_range)
+                    .map(CellValue::Date)
             }
             .unwrap_or(CellValue::String(v.to_owned()))
         } else if value.is_duration_iso() {

{python_calamine-0.4.0 → python_calamine-0.5.1}/src/types/sheet.rs

@@ -209,12 +209,15 @@ impl CalamineSheet {
             Arc::clone(&slf.range)
         };
 
-        PyList::new(
-            slf.py(),
-            range.rows().take(nrows as usize).map(|row| {
-                PyList::new(slf.py(), row.iter().map(<&Data as Into<CellValue>>::into)).unwrap()
-            }),
-        )
+        let py_list = PyList::empty(slf.py());
+
+        for row in range.rows().take(nrows as usize) {
+            let py_row = PyList::new(slf.py(), row.iter().map(<&Data as Into<CellValue>>::into))?;
+
+            py_list.append(py_row)?;
+        }
+
+        Ok(py_list)
     }
 
     fn iter_rows(&self) -> CalamineCellIterator {

{python_calamine-0.4.0 → python_calamine-0.5.1}/src/types/workbook.rs

@@ -102,7 +102,7 @@ impl CalamineWorkbook {
     fn py_from_object(
         _cls: &Bound<'_, PyType>,
         py: Python<'_>,
-        path_or_filelike: PyObject,
+        path_or_filelike: Py<PyAny>,
     ) -> PyResult<Self> {
         Self::from_object(py, path_or_filelike)
     }
@@ -112,22 +112,22 @@ impl CalamineWorkbook {
     fn py_from_filelike(
         _cls: &Bound<'_, PyType>,
         py: Python<'_>,
-        filelike: PyObject,
+        filelike: Py<PyAny>,
     ) -> PyResult<Self> {
-        py.allow_threads(|| Self::from_filelike(filelike))
+        py.detach(|| Self::from_filelike(filelike))
     }
 
     #[classmethod]
     #[pyo3(name = "from_path")]
-    fn py_from_path(_cls: &Bound<'_, PyType>, py: Python<'_>, path: PyObject) -> PyResult<Self> {
+    fn py_from_path(_cls: &Bound<'_, PyType>, py: Python<'_>, path: Py<PyAny>) -> PyResult<Self> {
         if let Ok(string_ref) = path.downcast_bound::<PyString>(py) {
             let path = string_ref.to_string_lossy().to_string();
-            return py.allow_threads(|| Self::from_path(&path));
+            return py.detach(|| Self::from_path(&path));
         }
 
         if let Ok(string_ref) = path.extract::<PathBuf>(py) {
             let path = string_ref.to_string_lossy().to_string();
-            return py.allow_threads(|| Self::from_path(&path));
+            return py.detach(|| Self::from_path(&path));
         }
 
         Err(PyTypeError::new_err(""))
@@ -135,12 +135,12 @@ impl CalamineWorkbook {
 
     #[pyo3(name = "get_sheet_by_name")]
     fn py_get_sheet_by_name(&mut self, py: Python<'_>, name: &str) -> PyResult<CalamineSheet> {
-        py.allow_threads(|| self.get_sheet_by_name(name))
+        py.detach(|| self.get_sheet_by_name(name))
     }
 
     #[pyo3(name = "get_sheet_by_index")]
     fn py_get_sheet_by_index(&mut self, py: Python<'_>, index: usize) -> PyResult<CalamineSheet> {
-        py.allow_threads(|| self.get_sheet_by_index(index))
+        py.detach(|| self.get_sheet_by_index(index))
     }
 
     fn close(&mut self) -> PyResult<()> {
@@ -160,30 +160,30 @@ impl CalamineWorkbook {
 
     fn __exit__(
         &mut self,
-        _exc_type: PyObject,
-        _exc_value: PyObject,
-        _traceback: PyObject,
+        _exc_type: Py<PyAny>,
+        _exc_value: Py<PyAny>,
+        _traceback: Py<PyAny>,
     ) -> PyResult<()> {
         self.close()
     }
 }
 
 impl CalamineWorkbook {
-    pub fn from_object(py: Python<'_>, path_or_filelike: PyObject) -> PyResult<Self> {
+    pub fn from_object(py: Python<'_>, path_or_filelike: Py<PyAny>) -> PyResult<Self> {
         if let Ok(string_ref) = path_or_filelike.downcast_bound::<PyString>(py) {
             let path = string_ref.to_string_lossy().to_string();
-            return py.allow_threads(|| Self::from_path(&path));
+            return py.detach(|| Self::from_path(&path));
         }
 
         if let Ok(string_ref) = path_or_filelike.extract::<PathBuf>(py) {
             let path = string_ref.to_string_lossy().to_string();
-            return py.allow_threads(|| Self::from_path(&path));
+            return py.detach(|| Self::from_path(&path));
         }
 
-        py.allow_threads(|| Self::from_filelike(path_or_filelike))
+        py.detach(|| Self::from_filelike(path_or_filelike))
     }
 
-    pub fn from_filelike(filelike: PyObject) -> PyResult<Self> {
+    pub fn from_filelike(filelike: Py<PyAny>) -> PyResult<Self> {
         let mut buf = vec![];
         PyFileLikeObject::with_requirements(filelike, true, false, true, false)?
             .read_to_end(&mut buf)?;

{python_calamine-0.4.0 → python_calamine-0.5.1}/tests/test_base.py

@@ -375,3 +375,10 @@ def test_close_workbook_double():
 
     with pytest.raises(WorkbookClosed):
         reader.close()
+
+
+def test_datetime_overflow():
+    reader = CalamineWorkbook.from_path(PATH / "issue139.xlsx")
+
+    # 2994626.0 = 10099-01-01
+    assert reader.get_sheet_by_index(0).to_python() == [["date"], [2994626.0]]

{python_calamine-0.4.0 → python_calamine-0.5.1}/tests/test_pandas.py

@@ -1,8 +1,6 @@
-from importlib.metadata import version
 from pathlib import Path
 
 import pytest
-from packaging.version import Version, parse
 
 try:
     import pandas as pd
@@ -14,17 +12,6 @@ except ImportError:
 PATH = Path(__file__).parent / "data"
 
 
-@pytest.fixture(scope="module", autouse=True)
-def pandas_monkeypatch():
-    if parse(version("pandas")) >= Version("2.2.0"):
-        yield
-    else:
-        from python_calamine.pandas import pandas_monkeypatch
-
-        pandas_monkeypatch()
-        yield
-
-
 @pytest.mark.skipif(not pd, reason="pandas is required")
 def test_ods_pandas(expected_df_ods):
     result = pd.read_excel(PATH / "base.ods", sheet_name="Sheet1", engine="calamine")

python_calamine-0.4.0/Cargo.toml

@@ -1,27 +0,0 @@
-[package]
-name = "python-calamine"
-version = "0.4.0"
-authors = ["Dmitriy <dimastbk@proton.me>"]
-description = "Python binding for Rust's library for reading excel and odf file - calamine"
-edition = "2021"
-homepage = "https://github.com/dimastbk/python-calamine"
-license = "MIT"
-readme = "README.md"
-repository = "https://github.com/dimastbk/python-calamine"
-
-[lib]
-name = "python_calamine"
-crate-type = ["cdylib"]
-
-[dependencies]
-calamine = { git = "https://github.com/tafia/calamine.git", rev = "5cda5c734661f656bdfb7ac7a1b95e77bf89aa09", features = ["dates"] }
-pyo3 = { version = "0.25.1", features = [
-    "extension-module",
-    "chrono",
-    "generate-import-lib",
-] }
-chrono = { version = "0.4.38", features = ["serde"] }
-pyo3-file = "0.13.0"
-
-[build-dependencies]
-pyo3-build-config = "0.25.1"

python_calamine-0.4.0/python/python_calamine/pandas.py

@@ -1,136 +0,0 @@
-from __future__ import annotations
-
-from datetime import date, datetime, time, timedelta
-from importlib.metadata import version
-from typing import TYPE_CHECKING, Union, cast
-
-import pandas as pd
-from packaging.version import Version, parse
-from pandas._typing import Scalar
-from pandas.compat._optional import import_optional_dependency
-from pandas.core.shared_docs import _shared_docs
-from pandas.io.excel import ExcelFile
-from pandas.io.excel._base import (  # type:ignore[attr-defined] # missing in pandas-stubs
-    BaseExcelReader,
-)
-from pandas.util._decorators import (  # type:ignore[attr-defined] # missing in pandas-stubs
-    doc,
-)
-
-if TYPE_CHECKING:
-    from pandas._typing import FilePath, ReadBuffer, StorageOptions
-    from python_calamine import CalamineSheet, CalamineWorkbook
-
-_CellValueT = Union[int, float, str, bool, time, date, datetime, timedelta]
-
-
-PANDAS_VERSION = parse(version("pandas"))
-
-
-class CalamineExcelReader(BaseExcelReader):
-    book: CalamineWorkbook
-
-    @doc(storage_options=_shared_docs["storage_options"])
-    def __init__(
-        self,
-        filepath_or_buffer: FilePath | ReadBuffer[bytes],
-        storage_options: StorageOptions | None = None,
-        engine_kwargs: dict | None = None,
-    ) -> None:
-        """
-        Reader using calamine engine (xlsx/xls/xlsb/ods).
-
-        Parameters
-        ----------
-        filepath_or_buffer : str, path to be parsed or
-            an open readable stream.
-        {storage_options}
-        engine_kwargs : dict, optional
-            Arbitrary keyword arguments passed to excel engine.
-        """
-        import_optional_dependency("python_calamine")
-        if PANDAS_VERSION >= Version("2.2.0"):
-            raise ValueError("Pandas >= 2.2.0 has builtin support of calamine")
-        elif PANDAS_VERSION >= Version("2.1.0"):
-            super().__init__(
-                filepath_or_buffer,
-                storage_options=storage_options,
-                engine_kwargs=engine_kwargs,
-            )
-        elif PANDAS_VERSION >= Version("2.0.0"):
-            super().__init__(
-                filepath_or_buffer,
-                storage_options=storage_options,
-            )
-        else:
-            raise ValueError("Pandas >= 2 is only supported")
-
-    @property
-    def _workbook_class(self) -> type[CalamineWorkbook]:
-        from python_calamine import CalamineWorkbook
-
-        return CalamineWorkbook
-
-    def load_workbook(
-        self,
-        filepath_or_buffer: FilePath | ReadBuffer[bytes],
-        engine_kwargs: dict | None = None,
-    ) -> CalamineWorkbook:
-        from python_calamine import load_workbook
-
-        return load_workbook(filepath_or_buffer, **(engine_kwargs or {}))
-
-    @property
-    def sheet_names(self) -> list[str]:
-        from python_calamine import SheetTypeEnum
-
-        return [
-            sheet.name
-            for sheet in self.book.sheets_metadata
-            if sheet.typ == SheetTypeEnum.WorkSheet
-        ]
-
-    def get_sheet_by_name(self, name: str) -> CalamineSheet:
-        self.raise_if_bad_sheet_by_name(name)
-        return self.book.get_sheet_by_name(name)
-
-    def get_sheet_by_index(self, index: int) -> CalamineSheet:
-        self.raise_if_bad_sheet_by_index(index)
-        return self.book.get_sheet_by_index(index)
-
-    def get_sheet_data(
-        self, sheet: CalamineSheet, file_rows_needed: int | None = None
-    ) -> list[list[Scalar]]:
-        def _convert_cell(value: _CellValueT) -> Scalar:
-            if isinstance(value, float):
-                val = int(value)
-                if val == value:
-                    return val
-                else:
-                    return value
-            elif isinstance(value, date):
-                return pd.Timestamp(value)
-            elif isinstance(value, timedelta):
-                return pd.Timedelta(value)
-            elif isinstance(value, time):
-                # cast needed here because Scalar doesn't include datetime.time
-                return cast(Scalar, value)
-
-            return value
-
-        rows: list[list[_CellValueT]] = sheet.to_python(skip_empty_area=False)
-        data: list[list[Scalar]] = []
-
-        for row in rows:
-            data.append([_convert_cell(cell) for cell in row])
-            if file_rows_needed is not None and len(data) >= file_rows_needed:
-                break
-
-        return data
-
-
-def pandas_monkeypatch() -> None:
-    ExcelFile._engines = {  # type:ignore[attr-defined] # missing in pandas-stubs
-        "calamine": CalamineExcelReader,
-        **ExcelFile._engines,  # type:ignore[attr-defined] # missing in pandas-stubs
-    }