[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,10 +1,8 @@
 from __future__ import annotations
 
 import math
-from pathlib import Path
 from typing import (
     TYPE_CHECKING,
-    Any,
     Dict,
     Hashable,
     Iterable,
@@ -30,7 +28,6 @@
     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
@@ -151,7 +148,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,
@@ -163,7 +160,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)
@@ -187,13 +187,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(),
@@ -208,12 +209,15 @@ def open_virtual_dataset(
         )
         non_loadable_vars = set(ds_virtual.variables).difference(_loadable_vars)
         ds_loadable = xr.open_zarr(
-            manifest_store, consolidated=False, zarr_format=3, drop_variables=non_loadable_vars
+            manifest_store,
+            consolidated=False,
+            zarr_format=3,
+            drop_variables=non_loadable_vars,
         )
         ds_virtual = ds_virtual.drop_vars(_loadable_vars)
         ds = xr.merge([ds_virtual, ds_loadable])
         return ds
-        
+
     @staticmethod
     def _dataset_chunk_manifest(
         path: str,