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

BoundingBoxType的label_text想改成string类型,但是很多报错 #50

Closed
xinsuinizhuan opened this issue Oct 11, 2021 · 9 comments
Closed
Labels
question Further information is requested Windows

Comments

@xinsuinizhuan
Copy link

图片

图片

想自己给label_text赋值string,然后发现label_text是为NULL,没法赋值,所以想把label_text改为string

@DefTruth
Copy link
Owner

因为很多模型都用了原来的label_text,直接修改的话会导致不兼容,需要把用到box类的都修改了

@xinsuinizhuan
Copy link
Author

只是,因为这个自定义模型的话,类别是自定义的,我通过外面传进来文件读取,所以这个类别不能写死

@xinsuinizhuan
Copy link
Author

xinsuinizhuan commented Oct 12, 2021

因为很多模型都用了原来的label_text,直接修改的话会导致不兼容,需要把用到box类的都修改了
但是,现在我外面传进来读取的时候,给lable_txt赋值的时候,发现它是NULL,没法赋值。那应该怎么操作呢?
new的话,后面这个都得费劲巴拉的进行销毁,所以最好是改成string变量,不进行new.

@DefTruth
Copy link
Owner

DefTruth commented Oct 12, 2021

能把onnx文件和你的类别放上来?我尝试帮你改一改?还有是用到了lite.ai.toolkit中的哪个类?应该可以抠出来用的,你可以把这个仓库看做一个案例库,找一些和你的任务比较接近的进行魔改一下,应该可以很好地适应新的模型。这里的提供了挺多的应用案例,是我学习各个推理引擎时整理的。

@xinsuinizhuan
Copy link
Author

我弄成功了。
图片

在析构函数里面再销毁,news的

@DefTruth
Copy link
Owner

优秀~

@xinsuinizhuan
Copy link
Author

优秀~

图片
感谢。这样,就初始化的时候,new一次,销毁时销毁掉,不用每次检测的时候,box里面来new和delete

@xinsuinizhuan
Copy link
Author

你好,原来的析构函数是:
~YoloV5() override = default;
我这样修改后,是不是析构函数不会调用父类BasicOrtHandler中的析构函数,释放资源了?
初始化调用的父类的出初始化,但是析构的时候,没调用父类,父类中创建的new的资源感觉没有释放掉。

@DefTruth
Copy link
Owner

你好,原来的析构函数是: ~YoloV5() override = default; 我这样修改后,是不是析构函数不会调用父类BasicOrtHandler中的析构函数,释放资源了? 初始化调用的父类的出初始化,但是析构的时候,没调用父类,父类中创建的new的资源感觉没有释放掉。

应该会调用父类的析构函数,我之前有测试过。你可以打log看看,或者把这个释放资源的函数改到子类也可以,父类析构时会判断是不是空指针。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested Windows
Projects
None yet
Development

No branches or pull requests

2 participants