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

[WIP] Regression Benchmark #146

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

[WIP] Regression Benchmark #146

wants to merge 10 commits into from

Conversation

nbei
Copy link
Collaborator

@nbei nbei commented Oct 26, 2021

No description provided.

# c) ``Pass``: Successfully pass the regression test.
# 3. We should check the representation of the result value in metafile and the
# generated log table. (Related to convert_str_metric_value() function)
# 4. The matric name should be mapped to the standard one. Please check
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

matric? Maybe Metric?


def main():
args = parse_args()
get_log_files(args)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe data_dict = get_log_files(args)?

@zengyh1900 zengyh1900 added kind/enhancement refactoring, improving CI, template, etc. awaiting response priority/P1 high priority labels Oct 12, 2022
@zengyh1900 zengyh1900 added this to the 0.8.0 milestone Oct 12, 2022
@zengyh1900
Copy link
Collaborator

This feature has been supported in https://github.com/open-mmlab/mmgeneration/tree/dev-1.x/.dev_scripts
I think @LeoXing1996 can help to improve this pull request so that it can be merged into master branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement refactoring, improving CI, template, etc. priority/P1 high priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants