Usability improvements - Githubissues

jeylau commented 1 year ago

Minor cosmetic changes (move video widget out of the way, hide unused checkboxes, ...)
Disable "Save All..." option, avoiding reported issues after attempting to save images
Let user load config, data, and files via PushButtons: the "load data" button becomes disabled after loading annotations (while the "load config" button remains as a config may still be loaded multiple times to update the colormap)
Add interactive tutorial: it only appears upon first GUI launch, but may be viewed again later on by going to View > Launch Tutorial.
[x] Fix compat bugs with napari 0.4.18

deprecated-napari-hub-preview-bot[bot] commented 1 year ago

Preview page for your plugin is ready here: https://preview.napari-hub.org/DeepLabCut/napari-deeplabcut/78 Updated: 2023-07-12T11:57:38.728708

n-poulsen commented 1 year ago

Hi @jeylau! Here's some feedback on what I saw while testing:

I find it confusing to have the "Load data" and "Load config" buttons visible when I'm already labeling data (have a config and folder loaded). That means you can then "easily" load a 2nd data folder into napari, which is confusing (might be more confusing for people like me, who had only ever used napari through the deeplabcut GUI/CLI and not launched it on it's own; wasn't quite sure why they were there before reading the docs). Is this just me or should it be updated?

Could you also edit your description to indicate what has changed? From my testing I saw

And then I don't think these are absolutely necessary, but would be nice to have (and could be done in a future PR):

only show the walkthrough the first time napari-deeplabcut is open (and have a button to go through the walkthrough again in the GUI in case a new user starts the app)
when labeling multi-animal projects, the color scheme reference does not match what is plotted (the keypoints are colored according to individual, not type). it would be nice to update it to reflect this

DeepLabCut / napari-deeplabcut