Add function to get per-bodypart RMSE by n-poulsen · Pull Request #2315 · DeepLabCut/DeepLabCut

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@n-poulsen

Copy link

Contributor

Adds a function to get RMSE for each bodypart, for both single and multi-animal projects.

  • Default behavior does not change (does not evaluate per-bodypart)
  • Evaluates per-bodypart when calling evaluate_network(..., per_keypoint_evaluation=True) and outputs the results to a file in the evaluation-results folder named DLCscorer + "-keypoint-results.csv"
  • Tests the evaluation code using synthetic data
MMathisLab reacted with heart emoji

@n-poulsen n-poulsen requested review from MMathisLab and jeylau

July 19, 2023 08:08

@MMathisLab

Copy link

Member

MMathisLab commented

Jul 19, 2023

edited

Loading

Awesome, I think this is a really useful addition. Edit: great, covers maDLC and dlc, thanks!

@n-poulsen

Copy link

Contributor Author

Sure, I'll add the parameter to the functional tests

MMathisLab reacted with thumbs up emoji

@n-poulsen n-poulsen added the enhancement

New feature or request

label

Jul 19, 2023

@jeylau

Copy link

Contributor

jeylau commented

Jul 24, 2023

Looking great, thanks @n-poulsen!

MMathisLab reacted with thumbs up emoji

@MMathisLab MMathisLab merged commit 18890db into main

Jul 24, 2023

@MMathisLab MMathisLab deleted the per_keypoint_evaluation branch

July 24, 2023 09:46

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MMathisLab MMathisLab MMathisLab approved these changes

+1 more reviewer

@jeylau jeylau jeylau approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

enhancement

New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@n-poulsen @MMathisLab @jeylau