italy_yaml fix loading datetimes

parse_datetime is already called in make_group, so we mustn't use it
with main_group options. (otherwise we will call parse_datetime or
make_timedelta twice, which is incorrect. this caused me a very
infuriating debug session because i thought my fix for #1565 was somehow
buggy.)

Author: prandla

cmscontrib/loaders/italy_yaml.py

@@ -230,10 +230,10 @@ def get_contest(self):
 
         # Times
         main_group = {}
-        load(conf, main_group, ["start", "inizio"], conv=parse_datetime)
-        load(conf, main_group, ["stop", "fine"], conv=parse_datetime)
+        load(conf, main_group, ["start", "inizio"])
+        load(conf, main_group, ["stop", "fine"])
         load(conf, args, ["timezone"])
-        load(conf, main_group, ["per_user_time"], conv=make_timedelta)
+        load(conf, main_group, ["per_user_time"])
 
         # Limits
         load(conf, args, "max_submission_number")
@@ -243,8 +243,8 @@ def get_contest(self):
 
         # Analysis mode
         load(conf, main_group, "analysis_enabled")
-        load(conf, main_group, "analysis_start", conv=parse_datetime)
-        load(conf, main_group, "analysis_stop", conv=parse_datetime)
+        load(conf, main_group, "analysis_start")
+        load(conf, main_group, "analysis_stop")
 
         # Groups
         main_group_name: str | None = load(conf, None, "main_group")