Skip to content

[npu-tipc] Minimize coco datasets for npu tipc #8226

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

Merged
merged 2 commits into from
May 10, 2023

Conversation

YanhuiDua
Copy link
Contributor

@YanhuiDua YanhuiDua commented May 10, 2023

由于目前npu缺失了一些vision op, 例如deformable_conv2, depthwise_conv2d_transpose, generate_proposals, distribute_fpn_proposals等;1个epoch的训练时间为gpu的几十倍,无法支持建设npu tipc benchmask;

此PR临时性地将coco_tipc datasets中train_image的数量由64减少至8,来保证可接受的时间内正常的tipc功能验证;

后续npu kernel支持完成,会将此部分代码删除

@paddle-bot
Copy link

paddle-bot bot commented May 10, 2023

Thanks for your contribution!

if [ ${train_img_num} != ${exp_num} ];then
echo "Replace with npu tipc coco training annotations"
mv $REPO_ROOT_PATH/dataset/coco/annotations/instances_train2017.json $REPO_ROOT_PATH/dataset/coco/annotations/instances_train2017_bak.json
wget https://raw.githubusercontent.com/YanhuiDua/PaddleDetection/npu_tipc/dataset/coco/annotations/instances_train2017.json
Copy link
Contributor

Choose a reason for hiding this comment

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

这个地址可以放到BOS上面去

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@LokeZhou LokeZhou merged commit 33d8697 into PaddlePaddle:develop May 10, 2023
@YanhuiDua YanhuiDua deleted the update_coco_tipc_for_npu branch May 10, 2023 11:12
YanhuiDua added a commit to YanhuiDua/PaddleDetection that referenced this pull request May 10, 2023
* change coco dataset for npu tipc

* update link
lyuwenyu pushed a commit that referenced this pull request May 16, 2023
* change coco dataset for npu tipc

* update link
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.

4 participants