• tests/data/cameras/minimal_session/calibration.toml
• tests/data/videos/min_session_back.mp4
• tests/data/videos/min_session_backL.mp4
• tests/data/videos/min_session_mid.mp4
• tests/data/videos/min_session_midL.mp4
• tests/data/videos/min_session_side.mp4
• tests/data/videos/min_session_sideL.mp4
• tests/data/videos/min_session_top.mp4
• tests/data/videos/min_session_topL.mp4

• sleap/io/cameras.py (3 hunks)
• sleap/io/dataset.py (1 hunks)
• sleap/io/format/hdf5.py (3 hunks)
• tests/fixtures/datasets.py (1 hunks)
• tests/io/test_cameras.py (3 hunks)
• tests/io/test_dataset.py (2 hunks)

• sleap/io/dataset.py

tests/fixtures/datasets.py (1)

• 275-280: The new fixture multiview_min_session_labels loads a minimal session file and searches for videos in the specified directory. Ensure that the path to the .slp file and the video directory are correct and accessible. Also, verify that the loaded labels are used correctly in the tests where this fixture is applied.

tests/io/test_cameras.py (2)

• 66-66: The number of cameras in the camera cluster has changed from 4 to 8. Please verify if this change is intentional and aligns with the test data.

• 204-209: The keys of session_dict["camcorder_to_video_idx_map"] have been changed from integers to strings. This could potentially break other parts of the code that expect these keys to be integers. Please ensure that all references to these keys are updated accordingly.

sleap/io/format/hdf5.py (3)

• 86-91: The "sessions" key has been added to the loop for deserializing metadata from the HDF5 file. Ensure that all existing files that this function will be used on contain the "sessions" key or can handle its absence gracefully.

• 325-330: The "sessions" key has been added to the loop for serializing metadata into the HDF5 file. This change should be compatible with the deserialization changes made in lines 86-91.

• 343-346: There's a typo correction in the comment, changing "PredcitedInstances" to "PredictedInstances". This doesn't affect the code execution but improves readability.

sleap/io/cameras.py (3)

• 762-766: The __repr__ method has been updated to include the number of videos in the representation. This change provides more detailed information about the RecordingSession object, which can be useful for debugging and logging.

• 828-834: The keys of camcorder_to_video_idx_map are now being converted to strings before assignment. This change seems to be related to serialization/deserialization, as JSON objects only support string keys. However, this could potentially introduce type-related bugs elsewhere in the code if it's expected that these keys are integers. Please verify that all uses of camcorder_to_video_idx_map throughout the codebase have been updated to handle string keys.

- camcorder_to_video_idx_map[cam_idx] = video_idx
+ camcorder_to_video_idx_map[str(cam_idx)] = str(video_idx)

• 864-869: Similar to the previous comment, the keys of camcorder_to_video_idx_map are now being converted back to integers when retrieving Camcorder and Video objects. Again, please verify that all uses of camcorder_to_video_idx_map throughout the codebase have been updated to handle this change.

- camcorder = session.camera_cluster.cameras[cam_idx]
- video = videos_list[video_idx]
+ camcorder = session.camera_cluster.cameras[int(cam_idx)]
+ video = videos_list[int(video_idx)]

tests/io/test_dataset.py (1)

• 972-979: The function signature of test_save_labels_with_sessions has been updated to include a new parameter multiview_min_session_labels. Ensure that all calls to this function throughout the codebase have been updated to match the new signature.