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

Polish codes of basic benchmark class and update tensorflow version to 2.0 in ci #405

Merged
merged 3 commits into from
May 15, 2020

Conversation

Xreki
Copy link
Collaborator

@Xreki Xreki commented May 15, 2020

拆分自#308 ,将其中一些与计时方式修改无关的代码拆离出来,方便review。
这个PR主要包含以下工作:

  • 清理benchmark基类实现,移除无用的代码
  • 将性能测试的结果改成用json.dumps后输出,修改后输出如下:
{"framework": "paddle", "version": "0.0.0", "name": "abs", "device": "GPU", "speed": {"repeat": 1, "begin": 0, "end": 1, "total": 7.61103630065918}}
  • 更新CI上的tf版本,并修改了grep的匹配规则,只匹配api/tests目录下的.py文件。这个PR CI中匹配到的修改文件如下:
++ grep 'api/tests.*.py$'
+ HAS_MODIFIED_API_TEST=api/tests/main.py

@paddle-bot-old
Copy link

Thanks for your contribution! Please wait for the result of CI firstly.

@paddle-bot-old
Copy link

The PR's message can't be empty.

@Xreki Xreki requested a review from GaoWei8 May 15, 2020 08:22
@Xreki Xreki merged commit 48a22eb into PaddlePaddle:master May 15, 2020
@Xreki Xreki deleted the api/refine branch May 15, 2020 08:41
Xreki added a commit to Xreki/benchmark that referenced this pull request Oct 20, 2020
…o 2.0 in ci (PaddlePaddle#405)

* Add log and tar file to gitignore.

* Polish codes of paddle and tensorflow basic benchmark implementation, remove some unused codes.

* Update the tf version in CI and only check the modified .py file.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants