search

kubeedge / ianvs

Distributed Synergy AI Benchmarking
https://ianvs.readthedocs.io
Apache License 2.0
115 stars 46 forks source link

OSPP: Smart Coding benchmark suite: built on KubeEdge-lanvs #159

Closed safe-b closed 3 weeks ago

safe-b commented 3 weeks ago

This PR is the implementation of #98

safe-b commented 3 weeks ago

此外，还有一些 CI 问题有待解决，请参阅 https://github.com/kubeedge/ianvs/actions/runs/11524309022/job/32095357988?pr=159

例如

Run if [ "3.7" = "3.9" ]; then
************* Module core.testenvmanager.dataset.dataset
core/testenvmanager/dataset/dataset.py:22:0: C0301: Line too long (106/100) (line-too-long)
core/testenvmanager/dataset/dataset.py:22:0: E06[11](https://github.com/kubeedge/ianvs/actions/runs/11524309022/job/32095357988?pr=159#step:5:12): No name 'JsonlDataParse' in module 'sedna.datasources' (no-name-in-module)
core/testenvmanager/dataset/dataset.py:22:0: E0611: No name 'JSONMetaDataParse' in module 'sedna.datasources' (no-name-in-module)
core/testenvmanager/dataset/dataset.py:28:0: R0902: Too many instance attributes (9/7) (too-many-instance-attributes)
core/testenvmanager/dataset/dataset.py:19:0: W0611: Unused import json (unused-import)

-----------------------------------
Your code has been rated at 9.92/10

Error: Process completed with exit code 30.

已经按照 https://github.com/kubeedge/ianvs/actions/runs/11524309022/job/32095357988?pr=159 进行了修改，对dataset.py文件进行了补充

safe-b commented 3 weeks ago

新的 CI 错误见 https://github.com/kubeedge/ianvs/actions/runs/11529775773/job/32099094184?pr=159

Run if [ "3.7" = "3.9" ]; then
************* Module core.testenvmanager.dataset.dataset
core/testenvmanager/dataset/dataset.py:468:0: C0304: Final newline missing (missing-final-newline)

-----------------------------------
Your code has been rated at 9.99/10

Error: The operation was canceled.

明白，好像是我的尾行格式设置有误，操作系统格式问题，正在修改

safe-b commented 3 weeks ago

Reference in

相关proposal已经删除，后续提交仅更新代码

MooreZheng commented 3 weeks ago

/lgtm

MooreZheng commented 3 weeks ago

/approve

kubeedge-bot commented 3 weeks ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: hsj576, MooreZheng

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/kubeedge/ianvs/blob/main/OWNERS)~~ [MooreZheng] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment