Skip to content

Commit fc5518b

Browse files
Dana SinghDana Singh
authored andcommitted
#48 Fix order of arguments
- remove "rewrite" - in addressing a plyint message about `form_remap_dep`, I messed up the flow.cylc b/c the arguments were not in the right order (corrected!)
1 parent 6d44c5d commit fc5518b

File tree

2 files changed

+11
-11
lines changed

2 files changed

+11
-11
lines changed

Jinja2Filters/form_remap_dep.py

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -22,13 +22,13 @@
2222
from pathlib import Path
2323
import yaml
2424

25-
def rewrite_form_remap_dep(grid_type: str,
26-
temporal_type: str,
27-
chunk: str,
28-
pp_components_str: str,
29-
output_type: str,
30-
yamlfile: str,
31-
history_segment: str=None) -> str:
25+
def form_remap_dep(grid_type: str,
26+
temporal_type: str,
27+
chunk: str,
28+
pp_components_str: str,
29+
output_type: str,
30+
yamlfile: str,
31+
history_segment: str=None) -> str:
3232
"""
3333
Form the task parameter list based on the grid type,
3434
the temporal type, and the desired pp component(s)

flow.cylc

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -306,16 +306,16 @@ rename-split-to-pp-native<native>
306306
{% endif %}
307307

308308
{% if DO_REGRID %}
309-
{{ "regrid-xy" | form_remap_dep('temporal', PP_CHUNK_A, PP_COMPONENTS, 'ts', HISTORY_SEGMENT, YAML) }}
309+
{{ "regrid-xy" | form_remap_dep('temporal', PP_CHUNK_A, PP_COMPONENTS, 'ts', YAML, HISTORY_SEGMENT) }}
310310
{% if DO_TIMEAVGS %}
311-
{{ "regrid-xy" | form_remap_dep('temporal', PP_CHUNK_A, PP_COMPONENTS, 'av', HISTORY_SEGMENT, YAML) }}
311+
{{ "regrid-xy" | form_remap_dep('temporal', PP_CHUNK_A, PP_COMPONENTS, 'av', YAML, HISTORY_SEGMENT) }}
312312
{% endif %}
313313
{% endif %}
314314

315315
{% if DO_NATIVE %}
316-
{{ "native" | form_remap_dep('temporal', PP_CHUNK_A, PP_COMPONENTS, 'ts', HISTORY_SEGMENT, YAML) }}
316+
{{ "native" | form_remap_dep('temporal', PP_CHUNK_A, PP_COMPONENTS, 'ts', YAML, HISTORY_SEGMENT) }}
317317
{% if DO_TIMEAVGS %}
318-
{{ "native" | form_remap_dep('temporal', PP_CHUNK_A, PP_COMPONENTS, 'av', HISTORY_SEGMENT, YAML) }}
318+
{{ "native" | form_remap_dep('temporal', PP_CHUNK_A, PP_COMPONENTS, 'av', YAML, HISTORY_SEGMENT) }}
319319
{% endif %}
320320
{% endif %}
321321

0 commit comments

Comments
 (0)