-
Notifications
You must be signed in to change notification settings - Fork 122
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
problem about Inference? #52
Comments
@ranpin Did you solve the problem? |
No, I didn't. Nobady discussed with me. |
@ranpin Hi, I did a quick run following the README instruction, and there was no coming issue. Here is a screenshot using the predicted anno file. |
Yes, it has no problem running. My confusion is :
|
|
ok, thank you. But, there is still a problem. You say that It is standard coco format. But due to the result of generated json cat_coco.json, it's not the same as the standard coco format. the category is not ture, i think it's better to keep align as well as the other categories. |
The categories depend on the |
I find that cat/class_with_id.txt only clarify the categories. for example cat. Then, how can i modify related category id to "17 " and supercategory to "animal"? |
Currently, it doesn't support this feature, you can change it in the source code in playground/mmdet_sam/images2coco.py Line 63 in 531e1cf
playground/mmdet_sam/images2coco.py Line 64 in 531e1cf
|
ok, thank you. I have no problem. But I still hope that you can support it some day. This will bring great convenience to our ordinary users.Thanks again! |
In mmdet-sam I tried a the code you gave as an example of inference using DINO and SAM model and got the predicted json file. The last image is my result, the first problem is that I am not sure if it is working and getting the correct annotation and segmentation results? It looks like garbled code, I used utf-8 just. The second problem is that the json file obtained from this inference result is in standard coco format by default?
the following is my commands.
the results follows:
The text was updated successfully, but these errors were encountered: