Skip to content

Commit 6e05e02

Browse files
committed
config: better episode naming in error messages
1 parent 3f14ed2 commit 6e05e02

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

src/gazeMapper/config.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -268,27 +268,27 @@ def _check_planes_per_episode(self, strict_check) -> type_utils.ProblemDict:
268268
case annotation.Event.Trial:
269269
allow_one_plane = allow_more_than_one = True
270270
if not allow_one_plane:
271-
msg = f'No planes should be defined for a {e.value} episode. Remove entry, even if its empty.'
271+
msg = f'No planes should be defined for a {annotation.tooltip_map[e]}. Remove entry, even if its empty.'
272272
if strict_check:
273273
raise ValueError(msg)
274274
else:
275275
type_utils.merge_problem_dicts(problems, {'planes_per_episode': {e: msg}})
276276
elif not self.planes_per_episode[e]:
277-
msg = ('At least one' if allow_more_than_one else 'One')+f' plane should be defined for a {e.value} episode'
277+
msg = ('At least one' if allow_more_than_one else 'One')+f' plane should be defined for a {annotation.tooltip_map[e]}'
278278
if strict_check:
279279
raise ValueError(msg)
280280
else:
281281
type_utils.merge_problem_dicts(problems, {'planes_per_episode': {e: msg}})
282282
if not allow_more_than_one and len(self.planes_per_episode[e])>1:
283-
msg = f'Only one plane should be defined for a {e.value} episode'
283+
msg = f'Only one plane should be defined for a {annotation.tooltip_map[e]}'
284284
if strict_check:
285285
raise ValueError(msg)
286286
else:
287287
type_utils.merge_problem_dicts(problems, {'planes_per_episode': {e: msg}})
288288

289289
for e in self.episodes_to_code:
290290
if e not in self.planes_per_episode and e!=annotation.Event.Sync_Camera and (e==annotation.Event.Sync_ET_Data and self.get_cam_movement_for_et_sync_method=='plane'):
291-
msg = f'{e.value} episodes are set up to be coded and require an associated plane, but no plane(s) are defined in planes_per_episode for {e.value} episodes'
291+
msg = f'{annotation.tooltip_map[e]}s are set up to be coded and require an associated plane, but no plane(s) are defined in planes_per_episode for {annotation.tooltip_map[e]}s'
292292
if strict_check:
293293
raise ValueError(msg)
294294
else:
@@ -300,15 +300,15 @@ def _check_episodes_to_code(self, strict_check) -> type_utils.ProblemDict:
300300
if not self.episodes_to_code:
301301
type_utils.merge_problem_dicts(problems, {'episodes_to_code': 'At minimum one episode should be selected to be coded'})
302302
if annotation.Event.Sync_ET_Data in self.episodes_to_code and self.get_cam_movement_for_et_sync_method=='':
303-
type_utils.merge_problem_dicts(problems, {'episodes_to_code': f'{annotation.Event.Sync_ET_Data.name} should not be listed in the episodes to be coded if there is no method for plane synchronization (get_cam_movement_for_et_sync_method) specified.'})
303+
type_utils.merge_problem_dicts(problems, {'episodes_to_code': f'{annotation.tooltip_map[annotation.Event.Sync_ET_Data]} should not be listed in the episodes to be coded if there is no method for plane synchronization (get_cam_movement_for_et_sync_method) specified.'})
304304

305305
for e in self.planes_per_episode:
306306
if e not in self.episodes_to_code:
307307
if strict_check:
308-
raise ValueError(f'Plane(s) are defined in planes_per_episode for {e.name} episodes, but {e.name} episodes are not set up to be coded in episodes_to_code. Fix episodes_to_code.')
308+
raise ValueError(f'Plane(s) are defined in planes_per_episode for {annotation.tooltip_map[e]}s, but {annotation.tooltip_map[e]}s are not set up to be coded in episodes_to_code. Fix episodes_to_code.')
309309
else:
310-
type_utils.merge_problem_dicts(problems, {'episodes_to_code': f'Plane(s) are defined in planes_per_episode for {e.value} episodes, but {e.value} episodes are not set up to be coded'})
311-
type_utils.merge_problem_dicts(problems, {'planes_per_episode': {e: f'{e.value} episodes are not set up to be coded in episodes_to_code, so no plane(s) should be set up for {e.value} episodes.'}})
310+
type_utils.merge_problem_dicts(problems, {'episodes_to_code': f'Plane(s) are defined in planes_per_episode for {annotation.tooltip_map[e]}s, but {annotation.tooltip_map[e]}s are not set up to be coded'})
311+
type_utils.merge_problem_dicts(problems, {'planes_per_episode': {e: f'{annotation.tooltip_map[e]}s are not set up to be coded in episodes_to_code, so no plane(s) should be set up for {annotation.tooltip_map[e]}s.'}})
312312
return problems
313313

314314
def _check_auto_coding_setup(self, strict_check) -> type_utils.ProblemDict:

0 commit comments

Comments
 (0)