Skip to content

Commit fba8382

Browse files
committed
Add parameter inference notes
1 parent 48ee55b commit fba8382

File tree

2 files changed

+9
-1
lines changed

2 files changed

+9
-1
lines changed

tests/integration/template_weekly_legacy_3.0.0_comprehensive_v3.cfg

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,12 @@ constraint = "#expand constraint#"
44
dry_run = "#expand dry_run#"
55
environment_commands = "#expand environment_commands#"
66
fail_on_dependency_skip = True
7-
infer_path_parameters = False
7+
# This setting does not support backwards compatibility.
8+
# Its purpose is specifically to call attention to parameters
9+
# that zppy would otherwise just infer, perhaps making an incorrect inference.
10+
# Therefore, it is expected that a user using this setting would want to get an error
11+
# noting new parameters to add explicitly to their cfg.
12+
# infer_path_parameters = False
813
infer_section_parameters = False
914
input = #expand user_input_v3#/E3SMv3/#expand case_name#
1015
input_subdir = archive/atm/hist

zppy/defaults/default.ini

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,9 @@ grid = string(default="")
3131
# These two parameters enable zppy to infer path or section parameters.
3232
# This allows users to set fewer parameters, but with the risk of zppy choosing incorrect values for them.
3333
# Set to False for more transparency in path or section defintions.
34+
# Please note `infer_path_parameters = False` does NOT support backwards compatibility.
35+
# The expectation is the user will want an error message describing parameters they haven't explicitly defined.
36+
# So, as new parameters get added in successive zppy versions, they can update their cfgs accordingly.
3437
infer_path_parameters = boolean(default=True)
3538
infer_section_parameters = boolean(default=True)
3639
# The directory to be post-processed

0 commit comments

Comments
 (0)