`dandi organize` breaks `PoseEstimation.original_videos` paths

[h-mayorquin]

If a user carefully sets the original_videos field in an ndx-pose PoseEstimation container to match the relative path of their source video files, dandi organize will break them.

dandi organize --update-external-file-paths renames the ImageSeries.external_file paths but leaves PoseEstimation.original_videos untouched, silently breaking the connection between pose estimation data and its source video. Currently, _get_image_series() only scans for pynwb.image.ImageSeries objects in the VIDEO_FILE_MODULES and rename_nwb_external_files() only rewrites container.external_file, with no awareness of ndx-pose's PoseEstimation.original_videos field.

A possible fix: rename_nwb_external_files() already builds a mapping of old paths to new paths for each video file it renames. It could also scan for PoseEstimation containers in the NWB file and apply that same mapping to any original_videos entries that match an old path.

Would this make sense?

[yarikoptic]

Sound sensible, but also fragile going forward happen they add/rename more of nwb fields :-/ But I do not see any better way ATM, if you can send a PR accompanied with a test on some "simulated" case -- would be appreciated. Also we could check on some file in archive if there is any already.

[h-mayorquin]

Maybe a better way is encoding a direct reference to the ImageSeries instead of a free-form link:

rly/ndx-pose#56

Let me think a little bit longer through this.