Merge pull request #1 from BitBloomTech/use-tables

Replace h5py dependency with tables

Author: phil-brad

azure-pipelines.yml

@@ -0,0 +1,48 @@
+trigger:
+  branches:
+    include:
+      - main
+      - refs/tags/*
+
+pool:
+  vmImage: ubuntu-latest
+
+stages:
+  - stage: Build
+    jobs:
+      - job: Python314Build
+        timeoutInMinutes: 120
+        container: python:3.14
+        steps:
+          - checkout: self
+            fetchDepth: 0
+            fetchTags: true
+          - script: python -m venv $(Build.SourcesDirectory)/.venv
+            displayName: "Creating virtual environment"
+          - task: PipAuthenticate@1
+            inputs:
+              artifactFeeds: 'bitbloom-artifacts'
+              onlyAddExtraIndex: True
+          - script: |
+              source $(Build.SourcesDirectory)/.venv/bin/activate
+              pip install .[test,build]
+            displayName: "Installing pvlib"
+          - script: |
+              source $(Build.SourcesDirectory)/.venv/bin/activate
+              python -m pytest tests
+            displayName: "Running tests with pytest"
+          - task: TwineAuthenticate@1
+            inputs:
+              artifactFeed: "bitbloom-artifacts"
+            condition: and(succeeded(), startsWith(variables['Build.SourceBranch'], 'refs/tags/'))
+          - script: |
+              source $(Build.SourcesDirectory)/.venv/bin/activate
+              TAG="${BUILD_SOURCEBRANCH#refs/tags/}"
+              TAG="${TAG#v}"
+              export SETUPTOOLS_SCM_PRETEND_VERSION="$TAG"
+              echo "Using package version: $SETUPTOOLS_SCM_PRETEND_VERSION"
+              python -m pip install build
+              python -m build --sdist --outdir dist
+              python -m twine upload -r "bitbloom-artifacts" --config-file $(PYPIRC_PATH) dist/*.tar.gz
+            displayName: "Building and publishing sdist package"
+            condition: and(succeeded(), startsWith(variables['Build.SourceBranch'], 'refs/tags/'))

pvlib/clearsky.py

@@ -10,7 +10,7 @@
 import numpy as np
 import pandas as pd
 from scipy.linalg import hankel
-import h5py
+import tables
 
 from pvlib import atmosphere, tools
 from pvlib.tools import _degrees_to_index
@@ -206,8 +206,8 @@ def lookup_linke_turbidity(time, latitude, longitude, filepath=None,
     latitude_index = _degrees_to_index(latitude, coordinate='latitude')
     longitude_index = _degrees_to_index(longitude, coordinate='longitude')
 
-    with h5py.File(filepath, 'r') as lt_h5_file:
-        lts = lt_h5_file['LinkeTurbidity'][latitude_index, longitude_index]
+    with tables.open_file(filepath, mode='r') as lt_h5_file:
+        lts = lt_h5_file.root.LinkeTurbidity[latitude_index, longitude_index]
 
     if interp_turbidity:
         linke_turbidity = _interpolate_turbidity(lts, time)

pvlib/location.py

@@ -10,7 +10,7 @@
 
 import pandas as pd
 import pytz
-import h5py
+import tables
 
 from pvlib import solarposition, clearsky, atmosphere, irradiance
 from pvlib.tools import _degrees_to_index
@@ -494,8 +494,8 @@ def lookup_altitude(latitude, longitude):
     latitude_index = _degrees_to_index(latitude, coordinate='latitude')
     longitude_index = _degrees_to_index(longitude, coordinate='longitude')
 
-    with h5py.File(filepath, 'r') as alt_h5_file:
-        alt = alt_h5_file['Altitude'][latitude_index, longitude_index]
+    with tables.open_file(filepath, mode='r') as alt_h5_file:
+        alt = alt_h5_file.root.Altitude[latitude_index, longitude_index]
 
     # 255 is a special value that means nodata. Fallback to 0 if nodata.
     if alt == 255:

pyproject.toml

@@ -1,5 +1,5 @@
 [build-system]
-requires = ["setuptools>=70.1", "setuptools_scm>=6.2"]
+requires = ["setuptools>=70.1", "setuptools_scm>=6.2", "twine"]
 build-backend = "setuptools.build_meta"
 
 
@@ -16,10 +16,9 @@ dependencies = [
     'pytz',
     'requests',
     'scipy >= 1.7.2',
-    'h5py',
+    'tables',
     'tzdata', # ensure all timezones are available in all installs for consistency GH#2795
 ]
-license = "BSD-3-Clause"
 classifiers = [
     'Development Status :: 4 - Beta',
     'Operating System :: OS Independent',
@@ -78,7 +77,10 @@ test = [
     'pytest-remotedata',
     'packaging',
 ]
-all = ["pvlib[test,optional,doc]"]
+build = [
+    "twine"
+]
+all = ["pvlib[test,optional,doc,build]"]
 
 [project.urls]
 "Bug Tracker" = "https://github.com/pvlib/pvlib-python/issues"