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.

Sign up for GitHub

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

Jump to bottom

ResNet-50 should be called ResNet-50 across all docs #431

Closed
matthew-frank opened this issue Jan 5, 2021 · 3 comments
Closed

ResNet-50 should be called ResNet-50 across all docs #431

matthew-frank opened this issue Jan 5, 2021 · 3 comments
Assignees
@matthew-frank
Labels
object_detection object_detection benchmark

Comments

@matthew-frank
Copy link
Contributor

https://github.com/mlcommons/training/tree/master/object_detection#4-model uses the term "ResNet50" twice. We are trying to standardize terminology and usage across mlcommons. Please change these to "ResNet-50" (with a dash between ResNet and 50).
@johntran-nv johntran-nv added image_classification image_classification benchmark object_detection object_detection benchmark and removed image_classification image_classification benchmark labels Nov 8, 2022
@johntran-nv
Copy link
Contributor

@matthew-frank could you propose a PR to fix this?

@matthew-frank matthew-frank self-assigned this Dec 2, 2022
@matthew-frank matthew-frank mentioned this issue Dec 2, 2022
Merged
@matthew-frank
Copy link
Contributor Author

PR #603 is my proposed fix for the small amount of this problem that remains.

@hiwotadese
Copy link
Contributor

Closing this issue because the benchmark is dropped for v4.1 and beyond.

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

@matthew-frank matthew-frank

Labels
object_detection object_detection benchmark
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@matthew-frank @johntran-nv @hiwotadese