search

Kitware / dive

Media annotation and analysis tools for web and desktop. Get started at https://viame.kitware.com
https://kitware.github.io/dive
Apache License 2.0
81 stars 21 forks source link

Disable repeated video data delete confirmation dialogue in Desktop #994

Open mattdawkins opened 3 years ago

mattdawkins commented 3 years ago

When dealing with many sequences, deleting data is a pain with confirmation dialogues on every delete. The information in the current delete dialogue should maybe only appear once, on the first user delete, or would be better suited for ? tooltip next to the delete button the user could hover over?

subdavis commented 3 years ago

This is why we added a setting to turn of delete confirmation. Did you use that?

mattdawkins commented 3 years ago

I mean data deletion from the video list page, not detections

subdavis commented 3 years ago

Oh no, that should not be removed. This is a dangerously destructive operation and it should be tedious.

subdavis commented 3 years ago

Are you talking about desktop? Because web has checkboxes for multiple deletes.

mattdawkins commented 3 years ago

It is tedious and a complete turn off to advanced users in current form, after a user reads that once they don't need to be inundated with it every time. Another way to rectify would be to indicate that the data doesn't exist on disk anymore greyed out and it doesn't query on those, or possibly mass delete with checkmarks like in web. People (e.g. me) sometimes decide to move their data archive around on disk which might be comprised of multiple videos / subfolders.

subdavis commented 3 years ago

Let's talk about it at the next meeting.

In the mean time, if you really need to delete a bunch of stuff from desktop, you can just delete the folders from the DIVE_Projects directory and then use the scan button on the settings page to re-synchronize.

waxlamp commented 2 years ago

@subdavis, is there anything blocking us from implementing checkbox-style mass delete on desktop?

subdavis commented 2 years ago

Checkbox-style mass delete would not resolve this issue.

The ask is to also delete media from outside the DIVE project directory, which I'm very wary of doing.

waxlamp commented 2 years ago

Checkbox-style mass delete would not resolve this issue.

The ask is to also delete media from outside the DIVE project directory, which I'm very wary of doing.

I would agree with your wariness. Something halfway out-of-band might be good: remove the "deleted" items from DIVE's awareness, and generate a report of all filenames and give to the user so they can do an xargs | rm or equivalent by their own hand.