Skip to content

Commit f83efa9

Browse files
committed
improve settings error around camera sync points
1 parent 70c2acc commit f83efa9

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

src/gazeMapper/config.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -269,6 +269,8 @@ def _check_planes_per_episode(self, strict_check) -> type_utils.ProblemDict:
269269
allow_one_plane = allow_more_than_one = True
270270
if not allow_one_plane:
271271
msg = f'No planes should be defined for a {annotation.tooltip_map[e]}. Remove entry, even if its empty.'
272+
if e==annotation.Event.Sync_ET_Data:
273+
msg += ' Alternatively, you may want to set the get_cam_movement_for_et_sync_method on the main options panel to "Plane".'
272274
if strict_check:
273275
raise ValueError(msg)
274276
else:
@@ -402,8 +404,8 @@ def _check_sync_ref(self, strict_check):
402404
if strict_check:
403405
raise ValueError('when sync_ref_recording is set, coding of camera sync points should be set up in episodes_to_code')
404406
else:
405-
problems['episodes_to_code'] = f'if sync_ref_recording is set, {annotation.Event.Sync_Camera.value} events should be set up to be coded'
406-
type_utils.merge_problem_dicts(problems, {'sync_ref_recording': f'sync_ref_recording is set, but {annotation.Event.Sync_Camera.value} events are not set up to be coded in episodes_to_code'})
407+
problems['episodes_to_code'] = f'if sync_ref_recording is set, {annotation.tooltip_map[annotation.Event.Sync_Camera]}s should be set up to be coded'
408+
type_utils.merge_problem_dicts(problems, {'sync_ref_recording': f'sync_ref_recording is set, but {annotation.tooltip_map[annotation.Event.Sync_Camera]}s are not set up to be coded in episodes_to_code'})
407409
return problems
408410

409411
def _check_et_sync_method(self, strict_check) -> type_utils.ProblemDict:

0 commit comments

Comments
 (0)