Skip to content
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

Add all-in-one graphic with all methods #613

Open
ioctl-user opened this issue Apr 29, 2021 · 3 comments
Open

Add all-in-one graphic with all methods #613

ioctl-user opened this issue Apr 29, 2021 · 3 comments
Labels
community/help wanted extra attention is needed enhancement New feature or request

Comments

@ioctl-user
Copy link

There are a lot of methods this is difficult to compare, so I have the following suggestion:

Add graphical picture for each data set with all methods with two axis: FPS and AP
Something like this: https://paperswithcode.com/sota/pose-estimation-on-coco-test-dev

@innerlee innerlee added the enhancement New feature or request label Apr 29, 2021
@innerlee
Copy link
Contributor

Thanks for the suggestion. We have plans for fancy modelzoo visualizations (including comparisons/filtrations/sorting/etc.). We are waiting for front-end programmers 😄
If anyone one can help, then we can bring them earlier.

@ioctl-user
Copy link
Author

Thanks!

As an additional suggestion:
Add one more metric to existing tables, that indicate stability of the top_down methods depending on box detector quality.
I.e., dependence of a given method AP quality from a box AP quality.

That may be useful for performance optimization, because one can prefer take fast box detector and steady pose detector.

@innerlee
Copy link
Contributor

innerlee commented Jun 2, 2021

@ioctl-user Please checkout the new modelzoo!

@ly015 ly015 added community/help wanted extra attention is needed and removed Community help wanted labels Oct 28, 2022
HAOCHENYE pushed a commit to HAOCHENYE/mmpose that referenced this issue Jun 27, 2023
* fix add_config func is not called bug
fix wandb add_config bug
add wandb log_code feature

* move log_code_name param to the last one
add log_code_name param to docstring

* add config only when there is a cfg

Co-authored-by: Zaida Zhou <[email protected]>

* add unit test for log_code_name param of WandbVisBackend

* Update mmengine/visualization/vis_backend.py

Co-authored-by: Zaida Zhou <[email protected]>

* use log_code_kwargs instead of only log_code_name
add PATH_FILTERS registry

* use log_code_kwargs instead of only log_code_name
add PATH_FILTERS registry

* fix add config unit test

* roll back to log_code_name version

Co-authored-by: Zaida Zhou <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community/help wanted extra attention is needed enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants