[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

virtualizarr/common.py

@@ -60,8 +60,7 @@ def replace_virtual_with_loadable_vars(
         decode_times=decode_times,
     ) as loadable_ds:
         var_names_to_load = get_loadable_variables(
-            dataset=loadable_ds,
-            loadable_variables=loadable_variables
+            dataset=loadable_ds, loadable_variables=loadable_variables
         )
         # this will automatically keep any IndexVariables needed for loadable 1D coordinates
         loadable_var_names_to_drop = set(loadable_ds.variables).difference(
@@ -83,8 +82,9 @@ def replace_virtual_with_loadable_vars(
             ],
         )
 
-def get_loadable_variables( 
-    dataset: xr.Dataset, 
+
+def get_loadable_variables(
+    dataset: xr.Dataset,
     loadable_variables: Iterable[Hashable] | None = None,
 ) -> Iterable[Hashable]:
     var_names_to_load: list[Hashable]
@@ -97,18 +97,14 @@ def get_loadable_variables(
         # coordinate variables.  We already have all the indexes and variables
         # we should be keeping - we just need to distinguish them.
         var_names_to_load = [
-            name
-            for name, var in dataset.variables.items()
-            if var.dims == (name,)
+            name for name, var in dataset.variables.items() if var.dims == (name,)
         ]
     else:
         raise ValueError(
             "loadable_variables must be an iterable of string variable names,"
             f" or None, but got type {type(loadable_variables)}"
         )
-    non_loadable_vars = set(dataset.variables).difference(
-        var_names_to_load
-    )
+    non_loadable_vars = set(dataset.variables).difference(var_names_to_load)
     return var_names_to_load
 
 

virtualizarr/readers/hdf/hdf.py

@@ -1,7 +1,6 @@
 from __future__ import annotations
 
 import math
-from pathlib import Path
 from typing import (
     TYPE_CHECKING,
     Hashable,
@@ -25,13 +24,11 @@
     ManifestGroup,
     ManifestStore,
 )
-from virtualizarr.manifests.manifest import validate_and_normalize_path_to_uri
 from virtualizarr.manifests.utils import create_v3_array_metadata
 from virtualizarr.readers.api import VirtualBackend
-from virtualizarr.readers.hdf.filters import cfcodec_from_dataset, codecs_from_dataset
+from virtualizarr.readers.hdf.filters import codecs_from_dataset
 from virtualizarr.types import ChunkKey
-from virtualizarr.utils import _FsspecFSFromFilepath, soft_import
-
+from virtualizarr.utils import soft_import
 
 h5py = soft_import("h5py", "For reading hdf files", strict=False)
 
@@ -85,21 +82,23 @@ def _construct_manifest_array(
         # Temporarily disable use CF->Codecs - TODO re-enable in subsequent PR.
         # cfcodec = cfcodec_from_dataset(dataset)
         # if cfcodec:
-            # codecs.insert(0, cfcodec["codec"])
-            # dtype = cfcodec["target_dtype"]
-            # attrs.pop("scale_factor", None)
-            # attrs.pop("add_offset", None)
+        # codecs.insert(0, cfcodec["codec"])
+        # dtype = cfcodec["target_dtype"]
+        # attrs.pop("scale_factor", None)
+        # attrs.pop("add_offset", None)
         # else:
-            # dtype = dataset.dtype
+        # dtype = dataset.dtype
 
         if "_FillValue" in attrs:
-            encoded_cf_fill_value = HDFVirtualBackend._encode_cf_fill_value(attrs["_FillValue"], dtype)
+            encoded_cf_fill_value = HDFVirtualBackend._encode_cf_fill_value(
+                attrs["_FillValue"], dtype
+            )
             attrs["_FillValue"] = encoded_cf_fill_value
 
         codec_configs = [
             numcodec_config_to_configurable(codec.get_config()) for codec in codecs
         ]
-        
+
         fill_value = dataset.fillvalue.item()
         dims = tuple(HDFVirtualBackend._dataset_dims(dataset, group=group))
         metadata = create_v3_array_metadata(
@@ -111,7 +110,6 @@ def _construct_manifest_array(
             dimension_names=dims,
             attributes=attrs,
         )
-        
 
         manifest = HDFVirtualBackend._dataset_chunk_manifest(path, dataset)
         return ManifestArray(metadata=metadata, chunkmanifest=manifest)
@@ -161,7 +159,7 @@ def _construct_manifest_group(
                     if variable is not None:
                         manifest_dict[key] = variable
         return ManifestGroup(arrays=manifest_dict, attributes=attrs)
-        
+
     @staticmethod
     def _create_manifest_store(
         filepath: str,
@@ -173,7 +171,10 @@ def _create_manifest_store(
     ) -> ManifestStore:
         # Create a group containing dataset level metadata and all the manifest arrays
         manifest_group = HDFVirtualBackend._construct_manifest_group(
-            store=store, filepath=filepath, group=group, drop_variables=drop_variables,
+            store=store,
+            filepath=filepath,
+            group=group,
+            drop_variables=drop_variables,
         )
         # Convert to a manifest store
         return ManifestStore(stores={prefix: store}, group=manifest_group)
@@ -197,13 +198,14 @@ def open_virtual_dataset(
             )
 
         # filepath = validate_and_normalize_path_to_uri(
-            # filepath, fs_root=Path.cwd().as_uri()
+        # filepath, fs_root=Path.cwd().as_uri()
         # )
 
         _drop_vars: list[Hashable] = (
             [] if drop_variables is None else list(drop_variables)
         )
         from obstore.store import LocalStore
+
         manifest_store = HDFVirtualBackend._create_manifest_store(
             filepath=filepath,
             store=LocalStore(),
@@ -212,11 +214,10 @@ def open_virtual_dataset(
             group=group,
         )
         ds = manifest_store.to_virtual_dataset(
-            loadable_variables=loadable_variables,
-            decode_times=decode_times
+            loadable_variables=loadable_variables, decode_times=decode_times
         )
         return ds
-        
+
     @staticmethod
     def _dataset_chunk_manifest(
         path: str,

virtualizarr/tests/test_readers/test_hdf/test_hdf.py

@@ -161,6 +161,7 @@ def test_multiple_attributes(self, string_attributes_hdf5_file):
         attrs = HDFVirtualBackend._extract_attrs(ds)
         assert len(attrs.keys()) == 2
 
+
 @requires_hdf5plugin
 @requires_imagecodecs
 class TestManifestGroupFromHDF: