Skip to content

Clip retirement#258

Merged
Sean1572 merged 25 commits intomainfrom
clip-retirement
Jun 14, 2022
Merged

Clip retirement#258
Sean1572 merged 25 commits intomainfrom
clip-retirement

Conversation

@Sean1572
Copy link
Collaborator

@Sean1572 Sean1572 commented Dec 20, 2021

Created a clip retirement system! Based on the precision between users, Pyrenote will automatically retire clips with high degrees of overlap. This allows users to annotate previously poorly labeled clips in the attempt in improving metrics

Added

  • New system for recommending clips!
    • IOU based clip retirement based on number of annotators and/or precision of their overlaps
    • Added controls to iou retirement to edit projects forms

Testing:

  • Create a new project, be sure to enable iou via Enable Experimental Quality Control in the rename menu and set desired max_user count and confidence threshold for retirements. (for testing, it helps to turn on example project to speed up testing)
  • Label the data with an admin account
  • Go to a new browser (recommended) and create a new account
  • Observe the clips you annotated as an admin appear in pending list in data with the new user. For the admin users, they will appeared as completed
  • With the new user, label some clips as you did before, and then label other clips poorly
  • Create a second new user, and open the project. Observe the clips you labeled well do not appear in pending, the clips labeled poorly will appear for this second users
  • In the admin account, try changing the max_user count and threshold limits. Observe changes for the second new user in the pending list
  • Repeat this process until all clips are retired

TODO

  • Check to make sure that the quick load data button and other data viewing types (all, marked for review, etc) are working normally
  • Add a retired clips view on frontend in the data portal
    • Make sure this is limited to admins only
  • add a data column in sql tables that denotes which clips are considered "retired" for data results purposes
  • Add counted label to csv

BUGS

  • Make sure that on systems without IOU metrics turned on, the data appears in the data portal

Code allows person to go through and annotate, slowly reducing what they can do
It works! Retires clips after 2 people check it if it hits 75% accuracy. Things to consider: 2 users may be too few, perhaps 3. Next: I need to also take away annotations other users have no done for projects running iou metrics
Shows clips the users has never touched and clips that the user has completed themselves
recently annotated clips will be marked 1, older clips will be marked as 2
Added power to change the given threshold, set a max limit for number of users to see the data, and included the number of views + confidence information on data portal (for now, for debuging purposes)
Displays the current confidence threshold and max user limit in the edit project form
@Sean1572 Sean1572 linked an issue Dec 20, 2021 that may be closed by this pull request
Added confidence and retirement information to csv. Also added counted status to csv. (Side note: fixed extra column being added by datetime object)
There was issues using next buttons with a diffrent active other than pending on an IOU project. Fixed this issue so active works as expected
Fixed bugs that appeared during demo today and made confidence metric run during saving
Prevented empty overlap dataframes from breaking apicalls with an if statement. Also changed manual_id from bird to labelCate.value. NOTE: THIS WILL BREAK FOR MUTLICLASS LABELS, NEED TO ADD IN THAT FUNCTIONALITY
Made note of mutliple-select labels
Sean1572 added 3 commits May 1, 2022 21:21
* Replaced IOU with Pairwise

Compare IOU with each user instead of grouping all the old users into one group.

* Removed old debug and added new note

* Reworked pairwise matrix

* increased timeoue

* Fixed 4 line bug

* Added IOU CSV Storage to SQL

Added
- A row in table to store iou score materix as pandas
- ability to download these csvs as a zip folder for more data anaysis
-Yeah
Put an general error handler around the clip confidence work just so clip successfully saves
Fixed bug were no Relevant Audio button didn't save the clip
Copy link
Member

@sprestrelski sprestrelski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@Sean1572 Sean1572 merged commit 735b532 into main Jun 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: high High Priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clip Retirement Bells and Whistles Change example to only show annotated labels to admins + have users not see other annotations from other users

2 participants