-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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
Support OneFormer #10714
base: dev-3.x
Are you sure you want to change the base?
Support OneFormer #10714
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
5a87447
to
4c84904
Compare
您好,正好这几天打算在我的baseline上复现oneformer, 我比较仔细地分析了您上传的代码, 我比较不能理解的是您在“oneformer_head.py”文件中, 似乎没有没有在forward函数中对text_tokenizer进行处理,并且没有找到其他位置对其进行处理,这是由于疏忽还是其他原因?从您给出的结果上看似乎和OneFormer官方几乎没有差异,如果代码无误,能解答一下text_tokenizer具体实现的位置吗?十分感谢 |
感谢您的关注,非常抱歉这部分工作因为考虑到代码复用性做了重构,在mmdet已复现的mask2former的基础上进行实现的,目前只做到对齐了推理精度,而上面提到的训练精度对齐是在之前版本的代码上做的,基本上是采用了oneformer算法的实现方式,但是由于git版本回退,那部分commit覆盖掉了。是我的疏忽,感谢您的指正。text_tokenizer需要在训练部分对标签得到的text文本进行处理与oneformer源码应该基本一致。 |
我尝试使用您上传的代码,并用此重新搭建了一个虚拟环境,但是始终无法进行训练, 我进行了很多尝试都没有解决这个问题
当我按照projects/SparseInst的示例中进行训练却可以正常跑通,我对比了SparseInst和OneFormer目录下的代码结构,也不认为OneFormer目录下的代码有问题,但是似乎projects.OneFormer.oneformer 始终无法正常导入, 这是我非常不能理解的 请问您那边使用“hyc_/oneformer”分支的代码能否正常导入OneFormer模块?如果可以,能否提供一份代码给我?感谢🙏 |
非常抱歉,因为一些原因我也找不到原始版本代码了,能找到的代码都在这个pr里了,看您的报错是路径问题,或许您可以尝试使用 |
…t on 21.02.2024
OneFormer(CVPR2023): https://github.com/SHI-Labs/OneFormer
The instance segmentation label is aligned with the oneformer official label, using the instance label converted from the panorama segmentation,For details, refer to https://github.com/SHI-Labs/OneFormer/blob/main/datasets/prepare_coco_semantic_annos_from_panoptic_annos.py