8000 Onboarding/workflow UX by talmo · Pull Request #484 · talmolab/sleap · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Onboarding/workflow UX #484

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 30 commits into from
Closed

Onboarding/workflow UX #484

wants to merge 30 commits into from

Conversation

talmo
Copy link
Collaborator
@talmo talmo commented Feb 22, 2021

Description

This PR adds a new set of GUI elements intended to guide users through the core workflow. This is intended to supplant wordy documentation guides on the website, keeping all GUI-only information actually within the GUI.

  • SLEAP icon
  • Splash widget: New, Open, Inference
  • Main panel to the dock with tabs associated with each of the main labeling steps
  • Guide panel with auto-detection of each workflow step
  • Simpler training widget to add as a tab
  • Standalone inference widget
  • Instance list panel (HUD is tricky to make editable/sortable)
  • Tracks list (especially so you can edit track names)
  • Proofreading mode (detect by seeing if project is all predictions with no suggestions and single video) -> shouldn't show core labeling workflow since this is the step where user fixes tracking
  • Display tip panel with GIF demonstrating how to drag nodes, mark them as not visible, rotate, etc

Types of changes

  • Bugfix
  • New feature
  • Refactor / Code style update (no logical changes)
  • Build / CI changes
  • Documentation Update
  • Other (explain)

Does this address any currently open issues?

No (?)

Thank you for contributing to SLEAP!

❤️

talmo added 29 commits February 24, 2021 20:22
- Rename Suggestions -> Labeling Queue and re-order columns
- Add resizing options to tables
- Suggestions generation defaults to "sample"
- Greedy check for instances instead of looping through everything
- Panels are now tabs in main dock
- Add guide widget for navigation/status check
- Add text descriptions to several of the panels
- Try to use training_config.json when searching for models
- use initial_config.json timestamp if not parseable from run name
- cache metrics instead of loading every time
- Add sort by string property to set some columns as sortable by string
- Separate sort function by column for overriding more easily
- Add state to data model
- Emit items_changed signal
- Fix model reset not being emitted when data tables are emptied
- Add is_enabled function to override item-wise enabled flag check
- Add GenericCheckableTableModel for tables with a checkbox for each row
- Add resize_mode as a property to GenericTableView
arie-matsliah added a commit that referenced this pull request Feb 25, 2021
* Pulling finished work into a separate PR

* fix test

* fix

* Simplify + tweak background
arie-matsliah added a commit that referenced this pull request Feb 25, 2021
arie-matsliah added a commit that referenced this pull request Feb 26, 2021
arie-matsliah added a commit that referenced this pull request Mar 5, 2021
arie-matsliah added a commit that referenced this pull request Mar 8, 2021
* Merge main back into develop (#491)

* Prep for master->main (#475)

Co-authored-by: Arie Matsliah <arie.matsliah@gmail.com>

* Release version 1.1.0

* Remove appveyor

Co-authored-by: Talmo Pereira <talmo@princeton.edu>
Co-authored-by: Arie Matsliah <arie.matsliah@gmail.com>

* Merge main to develop post release (#492)

* Prep for master->main (#475)

Co-authored-by: Arie Matsliah <arie.matsliah@gmail.com>

* Release version 1.1.0

* Remove appveyor

* Test docs subfolders

* Test keep_files

Co-authored-by: Talmo Pereira <talmo@princeton.edu>
Co-authored-by: Arie Matsliah <arie.matsliah@gmail.com>

* Pulling finished work from #484 into a separate PR (#493)

* Pulling finished work into a separate PR

* fix test

* fix

* Simplify + tweak background

* Update docs (#495)

* Isolated fix for issue #480 (#498)

* Add support for specifying the minimum score in PAF matching (#502)

Effective at filtering out explicitly spurious matches when no better match is available

* Fix post-training errors (#503)

- Catch filesystem busy error when deleting viz dir
- Don't throw an error after training with no inference target

* Revert "Pulling finished work from #484 into a separate PR" (#504)

* Minor fixes and enhacements (#505)

- Add skeleton.find_neighbors convenience graph op
- Add Instance.fill_missing to initialize new nodes in the skeleton
- Also add numpy() and from_pointsarray() aliases to PredictedInstance
  to prevent crashes from using the inherited versions
- Initialize missing instance nodes when plotting user instances
- Fix serialization after loading MediaVideo from high level API
- Fix single instance visualization during training
- Add --open-in-gui CLI option to sleap-track
- Clearer error message when running sleap-track without args

* [WIP] Fix multi size video support (#501)

* wip

* cleanup

* Add `max_height_and_width` property to `VideoReader`

* Fix SizeMatcher instantiation without specified targets
- Lint/line length

* Provider now recreates the data pipeline with new providers

* Vectorize padding test for SizeMatcher

* Do not require instances key if not available from provider

* Size matching

* Size matching fix

* Fix shapes in output dataset

* lint

Co-authored-by: Talmo Pereira <talmo@princeton.edu>

* Bump version for release (#506)

Co-authored-by: Talmo Pereira <talmo@princeton.edu>
Co-authored-by: Arie Matsliah <arie.matsliah@gmail.com>
arie-matsliah added a commit that referenced this pull request Mar 8, 2021
arie-matsliah added a commit that referenced this pull request Mar 29, 2021
< 8000 div class="AvatarStack-body" > @arie-matsliah
arie-matsliah added a commit that referenced this pull request Apr 26, 2021
@talmo talmo mentioned this pull request May 12, 2022
@talmo
Copy link
Collaborator Author
talmo commented May 12, 2022

Closing this one, but follow #733 for progress on this issue.

@talmo talmo closed this May 12, 2022
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
0