Skip to content

Commit cd2181b

Browse files
author
Ruban Hussain
committed
fix: small bug fix to print statements for update_list_of_dicts_with_values_from_config
1 parent 263594a commit cd2181b

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

src/sagemaker/utils.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1297,6 +1297,8 @@ def update_list_of_dicts_with_values_from_config(
12971297
return
12981298
inputs_copy = copy.deepcopy(input_list)
12991299
inputs_from_config = get_sagemaker_config_value(sagemaker_session, config_key_path) or []
1300+
unmodified_inputs_from_config = copy.deepcopy(inputs_from_config)
1301+
13001302
for i in range(min(len(input_list), len(inputs_from_config))):
13011303
dict_from_inputs = input_list[i]
13021304
dict_from_config = inputs_from_config[i]
@@ -1316,11 +1318,12 @@ def update_list_of_dicts_with_values_from_config(
13161318
# Don't do the merge, Union parameters are not obeyed.
13171319
continue
13181320
input_list[i] = dict_from_config
1319-
if inputs_from_config:
1321+
1322+
if unmodified_inputs_from_config:
13201323
print(
13211324
"[Sagemaker Config - applied value]\n",
13221325
"config key = {}\n".format(config_key_path),
1323-
"config value = {}\n".format(inputs_from_config),
1326+
"config value = {}\n".format(unmodified_inputs_from_config),
13241327
"source value = {}\n".format(inputs_copy),
13251328
"combined value that will be used = {}\n".format(input_list),
13261329
)

0 commit comments

Comments
 (0)