FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Allow same filenames in different directories for multicam import by mattdawkins · Pull Request #1579 · Kitware/dive · GitHub

/ dive Public

Allow same filenames in different directories for multicam import - #1579

Merged
BryonLewis merged 1 commit into
mainfrom
fix/allow-same-multi-cam-names
May 4, 2026
Merged

BryonLewis merged 1 commit into
mainfrom
fix/allow-same-multi-cam-names

Conversation

mattdawkins commented Jan 24, 2026
edited
Loading

Copy link
Copy Markdown
Member

Summary

Allow same filenames in different directories for stereo / multicam imports.

Changes vs main

  • Fix overlap validation in ImportMultiCamDialog.vue so it only forbids duplicate filenames in keyword/glob mode (where images come from a single shared directory). Multi-folder mode now permits same filenames across cameras (e.g. left/image001.png + right/image001.png), which is a valid stereo setup.

Previously, DIVE would error when importing stereo or multicamera data
where images in different directories had the same filenames (e.g.,
left/image001.png and right/image001.png). This is a valid use case
for stereo camera setups.

The fix modifies the overlap validation in ImportMultiCamDialog.vue to
only check for duplicate filenames in keyword/glob mode (where images
come from a single shared directory). In multi-folder mode, same
filenames in different directories are now allowed since each camera
has its own source directory.
mattdawkins force-pushed the fix/allow-same-multi-cam-names branch 2 times, most recently from 723d567 to e25fbef Compare April 29, 2026 09:44
BryonLewis merged commit cfb178e into main May 4, 2026
3 checks passed
BryonLewis deleted the fix/allow-same-multi-cam-names branch May 4, 2026 13:41
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL