make-sense icon indicating copy to clipboard operation
make-sense copied to clipboard

Delete an image from the dataset

Open scottamain opened this issue 4 years ago • 6 comments

Is your feature request related to a problem? Please describe. While annotating my dataset, I realize some images are garbage and don't want to use them.

Describe the solution you'd like If there are no labels, pressing delete will remove the image completely. (The delete key already works to delete a label, so all labels would have to first be deleted before you can remove the image.)

scottamain avatar Oct 18 '21 23:10 scottamain

Hello @scottamain 👋 ! Thank you for your interest in our project, as well as, for taking the time to create this issue. However, I'll close that issue, as we already have an issue covering this functionality https://github.com/SkalskiP/make-sense/issues/73. If in the future you'll see room for improvement of makesense.ai please let us know.

SkalskiP avatar Oct 21 '21 19:10 SkalskiP

Thanks for your response @SkalskiP, but I don't see any discussion in #73 about removing photos from the training set. Even if it did, I think the ability to delete one photo at a time is a feature that ought to be implemented as a discreet task rather as part of a much more complicated data augmentation tool. Would you please consider reopening this FR (even if as a sub-task for the augmentation feature)?

scottamain avatar Oct 22 '21 00:10 scottamain

@scottamain so in this task you are simply proposing to for example add a delete button to the top panel?

SkalskiP avatar Oct 22 '21 06:10 SkalskiP

Yeah, with the Delete key as the shortcut.

I think it would require a confirmation dialog ("Do you want to delete the original image from the source?") to avoid any accidents, such as if you are furiously deleting all annotations with the Delete key and hit it one extra time.

scottamain avatar Oct 25 '21 17:10 scottamain

In that case, let's reopen that issue and let's do it :) As for the extra dialog window. I think we will let users know about potential unwanted consequences. But I also think we will add a "Don't show me this information again" checkbox.

SkalskiP avatar Oct 25 '21 20:10 SkalskiP

Great, thank you!

scottamain avatar Oct 25 '21 22:10 scottamain