WeSign/sign-predictor
1
0
Fork 0
Code Issues Pull Requests 1 Packages Projects Releases 3 Wiki Activity

Basics sign predictor #17

Open
robdwael wants to merge 2 commits from WES-129-basic-sign-predictor-model into dev
pull from: WES-129-basic-sign-predictor-model
merge into: WeSign:dev
Conversation 3 Commits 2 Files Changed 6 +389 -1408
robdwael commented 2023-04-24 09:39:19 +00:00 (Migrated from gitlab.ilabt.imec.be)
Copy Link

Description

Created a script to create the basic signs dataset
Adapted train.py to train_basics.py to train SPOTER model on basic signs

Testing Instructions

Please provide instructions on how the code reviewers can test your changes:

Load the model in the model live view (visualizations/webcam_view.py) and/or analyze it using the analyze_model notebook

The performance is still quite bad and webcam_view has not been adapted to display the correct sign names

Related Issues

Known bugs or issues

Bad accuracy, low test size

Checklist

  • I have filled in this template.
  • I have tested my changes thoroughly.
  • I have updated the user documentation as necessary.
  • Code reviewed by 1 person.

Additional Notes

Basically useless now but I want this branch gone

Closes WES-129

## Description Created a script to create the basic signs dataset Adapted train.py to train_basics.py to train SPOTER model on basic signs ## Testing Instructions Please provide instructions on how the code reviewers can test your changes: Load the model in the model live view (visualizations/webcam_view.py) and/or analyze it using the analyze_model notebook The performance is still quite bad and webcam_view has not been adapted to display the correct sign names ## Related Issues ## Known bugs or issues Bad accuracy, low test size ## Checklist - [x] I have filled in this template. - [x] I have tested my changes thoroughly. - [x] I have updated the user documentation as necessary. - [ ] Code reviewed by 1 person. ## Additional Notes Basically useless now but I want this branch gone Closes WES-129
robdwael commented 2023-04-24 09:39:20 +00:00 (Migrated from gitlab.ilabt.imec.be)
Copy Link

requested review from @vimylle

requested review from @vimylle
robdwael commented 2023-04-24 09:40:28 +00:00 (Migrated from gitlab.ilabt.imec.be)
Copy Link

changed the description

changed the description
robdwael commented 2023-05-21 15:24:18 +00:00 (Migrated from gitlab.ilabt.imec.be)
Copy Link

changed the description

changed the description
This pull request has changes conflicting with the target branch.
  • visualizations/visualize_data.ipynb
  • visualizations/webcam_view.py
View command line instructions.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin WES-129-basic-sign-predictor-model:WES-129-basic-sign-predictor-model
git checkout WES-129-basic-sign-predictor-model
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
enhancement
impact::high
impact::low
impact::minor
impact::moderate
impact::very low
probability::high
probability::low
probability::minor
probability::moderate
probability::very low
progress::busy
progress::done
progress::review
progress::todo
version::0.1
version::0.2
version::0.3
version::0.4
version::0.5
version::1.0
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
VictorMylle
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: WeSign/sign-predictor#17
No description provided.
Delete Branch "WES-129-basic-sign-predictor-model"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?