Skip to content

Clip retirement#275

Merged
Sean1572 merged 32 commits intomainfrom
clip-retirement
Jun 27, 2022
Merged

Clip retirement#275
Sean1572 merged 32 commits intomainfrom
clip-retirement

Conversation

@Sean1572
Copy link
Collaborator

@Sean1572 Sean1572 commented Jun 24, 2022

Added bug fixes to clip retirement.

The big bug was with saving, triple check with all diffrent kinds of labels to ensure that Pyrenote can save without werid errors

When creating projects, click every button in the admin portal to make sure the settings are good. Be sure to enable next_button on the feature toggle popup and try doing 2 projects: 1 with the experimental quality control pipeline and test that out.

Test on mutliple users to make sure the projects are working as intended. IOU projects should allow people to redo the clips without showing annotations from other users. Normal projects do not do this

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
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
* 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
- Changed save all feature so it sends all segmentation data to the backend so it processes all at once as opposed to calling the api call mutliple times

- This improves proformance slightly with confidence work as now we don't need to run mutliple iou mertics for each indivual annotation
- Removed unneeded frontend console output

- Fixed issue where error message appears if save all does not finish when user moves to a new page

- Fixed issue where if a user moves to a new clip, batch saving does not work
Fixed issue where if a project did not have IOU control pipeline enabled, it would not show clips on the data pipeline
Fixed bug where quick start would take you to annotation page even if no data was ready to be annotated!
@Sean1572 Sean1572 added the priority: high High Priority label Jun 24, 2022
@sprestrelski sprestrelski self-requested a review June 27, 2022 06:15
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.

When testing saving, I was able to save with four different accounts accessing the same clip at once, and it seemed to show up properly according to parameters set for clip retirement.

IOU projects should allow people to redo the clips without showing annotations from other users. Normal projects do not do this

I'm not sure I understand this properly - in both IoU and normal projects, I cannot see spectrogram annotations from other users.

One interesting behavior I found is that if you

  1. make a normal project
  2. make annotations
  3. then, switch the project to have the IOU scores, it does not automatically load the annotations - users need to refresh the page and open each clip again in order for them to show as "annotated"

@Sean1572
Copy link
Collaborator Author

it does not automatically load the annotations - users need to refresh the page and open each clip again in order for them to show as "annotated"

This is consistent with normal behavior, awesome I think we are good to merge!

@Sean1572 Sean1572 merged commit d6e26a6 into main Jun 27, 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.

2 participants