Skip to content

Commit f2cb1a7

Browse files
[pre-commit.ci] auto fixes from pre-commit.com hooks
for more information, see https://pre-commit.ci
1 parent bde0771 commit f2cb1a7

File tree

5 files changed

+1
-11
lines changed

5 files changed

+1
-11
lines changed

builders/cmip.py

-1
Original file line numberDiff line numberDiff line change
@@ -202,7 +202,6 @@ def build_cmip(
202202
@click.option('-v', '--cmip-version', type=int, help='CMIP phase (e.g. 5 for CMIP5 or 6 for CMIP6)')
203203
@click.option('--csv-filepath', type=str, help='File path to use when saving the built catalog')
204204
def cli(root_path, depth, pick_latest_version, cmip_version, csv_filepath):
205-
206205
if cmip_version not in set([5, 6]):
207206
raise ValueError(f'cmip_version = {cmip_version} is not valid. Valid options include: 5 and 6.')
208207

builders/core.py

-1
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,6 @@ def reverse_filename_format(filename, templates):
5555

5656

5757
def extract_attr_with_regex(input_str, regex, strip_chars=None, ignore_case=True):
58-
5958
if ignore_case:
6059
pattern = re.compile(regex, re.IGNORECASE)
6160

builders/notebooks/aws-cordex_catalog_builder.ipynb

-2
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,6 @@
4545
"outputs": [],
4646
"source": [
4747
"def get_file_list(fs):\n",
48-
"\n",
4948
" # Remove 'catalogs' directory from top level listing to get frequencies\n",
5049
" frequencies = [f for f in fs.ls(root) if 'catalogs' not in f]\n",
5150
"\n",
@@ -124,7 +123,6 @@
124123
"outputs": [],
125124
"source": [
126125
"def get_filename_attrs(store, fs=fs):\n",
127-
"\n",
128126
" template = (\n",
129127
" \"s3://ncar-na-cordex/{frequency}/{variable}.{scenario}.{frequency}.{grid}.{bias_correction}.zarr\"\n",
130128
" )\n",

builders/notebooks/aws-dart_catalog_builder.ipynb

-2
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,6 @@
8484
"outputs": [],
8585
"source": [
8686
"def get_file_list(store_path):\n",
87-
"\n",
8887
" store_path_str = store_path.as_posix()\n",
8988
"\n",
9089
" # Remove 'catalogs' directory from top level listing to get frequencies\n",
@@ -171,7 +170,6 @@
171170
"source": [
172171
"# def get_filename_attrs(store, fs=fs):\n",
173172
"def get_filename_attrs(store):\n",
174-
"\n",
175173
" relative_path = os.path.relpath(store, start=zarr_dir)\n",
176174
" print(relative_path)\n",
177175
" template = \"{frequency}/{variable}.zarr\"\n",

catalogs/aws-cesm1-le.json

+1-5
Original file line numberDiff line numberDiff line change
@@ -51,11 +51,7 @@
5151
},
5252
"aggregation_control": {
5353
"variable_column_name": "variable",
54-
"groupby_attrs": [
55-
"component",
56-
"experiment",
57-
"frequency"
58-
],
54+
"groupby_attrs": ["component", "experiment", "frequency"],
5955
"aggregations": [
6056
{
6157
"type": "union",

0 commit comments

Comments
 (0)