The snapshot bot encapsulates a bunch of image preprocessing logic in ImageInput classes. These are not thread-safe e.g. m_Unwrapped at https://github.com/BrainsOnBoard/bob_robotics/blob/snapshot_bot_load_fix/projects/snapshot_bot/image_input.cc#L71 is reused to reduce runtime memory allocations. However, they were being applied during parallelized loading which caused very weird image corruption issues! Making these thread-safe would make this code very messy so I'm just running these serially.
The snapshot bot encapsulates a bunch of image preprocessing logic in
ImageInput
classes. These are not thread-safe e.g.m_Unwrapped
at https://github.com/BrainsOnBoard/bob_robotics/blob/snapshot_bot_load_fix/projects/snapshot_bot/image_input.cc#L71 is reused to reduce runtime memory allocations. However, they were being applied during parallelized loading which caused very weird image corruption issues! Making these thread-safe would make this code very messy so I'm just running these serially.