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

add Yolov7 model support #544

Closed
wants to merge 14 commits into from
Closed

add Yolov7 model support #544

wants to merge 14 commits into from

Conversation

kadirnar
Copy link
Contributor

@kadirnar kadirnar commented Aug 4, 2022

No description provided.

@kadirnar
Copy link
Contributor Author

kadirnar commented Aug 6, 2022

@fcakyon

Local de tüm testlerden geçti. Neden yolov5 dosyalarında yolov7 modelini arıyor?
Hata Mesaji:

TypeError: ('model_path is not a valid yolov7 model path: ', AttributeError("Can't get attribute 'MP' on <module 'models.common' from '/opt/hostedtoolcache/Python/3.6.15/x64/lib/python3.6/site-packages/yolov5/models/common.py'>",))

sahi/model.py Outdated Show resolved Hide resolved
sahi/model.py Show resolved Hide resolved
sahi/model.py Show resolved Hide resolved
sahi/model.py Show resolved Hide resolved
@fcakyon
Copy link
Collaborator

fcakyon commented Aug 11, 2022

Local de tüm testlerden geçti. Neden yolov5 dosyalarında yolov7 modelini arıyor? Hata Mesaji:

TypeError: ('model_path is not a valid yolov7 model path: ', AttributeError("Can't get attribute 'MP' on <module 'models.common' from '/opt/hostedtoolcache/Python/3.6.15/x64/lib/python3.6/site-packages/yolov5/models/common.py'>",))

CI teslerindeki bu hata cok ilginc degil mi @devrimcavusoglu ?

@kadirnar
Copy link
Contributor Author

Local de tüm testlerden geçti. Neden yolov5 dosyalarında yolov7 modelini arıyor? Hata Mesaji:

TypeError: ('model_path is not a valid yolov7 model path: ', AttributeError("Can't get attribute 'MP' on <module 'models.common' from '/opt/hostedtoolcache/Python/3.6.15/x64/lib/python3.6/site-packages/yolov5/models/common.py'>",))

CI teslerindeki bu hata cok ilginc degil mi @devrimcavusoglu ?

Hata düzeldi galiba fakat local şuan model None çıktısı veriyor. Fakat local de testlerden geçiyor.

@kadirnar
Copy link
Contributor Author

Local de tüm testlerden geçti. Neden yolov5 dosyalarında yolov7 modelini arıyor? Hata Mesaji:

TypeError: ('model_path is not a valid yolov7 model path: ', AttributeError("Can't get attribute 'MP' on <module 'models.common' from '/opt/hostedtoolcache/Python/3.6.15/x64/lib/python3.6/site-packages/yolov5/models/common.py'>",))

CI teslerindeki bu hata cok ilginc degil mi @devrimcavusoglu ?

Hata düzeldi galiba fakat local şuan model None çıktısı veriyor. Fakat local de testlerden geçiyor.

@devrimcavusoglu @fcakyon

Yeni env açıp test ettiğimde test kodlarındaki hatayi verdi. Test mesajlarına ek olarak no module pandas hatasını verdi. Test kodlarından geçebilmesi için requirements dosyasındaki kütüphanelerin yüklü olması gerekiyor. Test ederken bu kütüphaneleri yüklemesi için paket.yaml dosyalarının mı güncellenmesi gerekiyor? #501 (comment)

@fcakyon fcakyon added the enhancement New feature or request label Sep 4, 2022
@fcakyon
Copy link
Collaborator

fcakyon commented Oct 11, 2022

@kadirnar main brnachteki degisiklikleri bu branche manuel olarak kopyalayip commitlersen merge conflict olusur bunu duzeltmek cok zor.

onun yerine git fetch + git pull origin main yaparak alman gerekiyor degisiklikleri

@fcakyon
Copy link
Collaborator

fcakyon commented Oct 11, 2022

@kadirnar su anda ciddi merge conflict olustu kendin elinle kopyalayip commitledigin icin

@kadirnar
Copy link
Contributor Author

@fcakyon
SAHI kütüphanesi detaylı günceleme almış. Manuel değişiklik yapmam baya hatalı olmuş. .yml kısmında birkaç manuel düzenleme de yaptım. Hatalara göre düzenleme yapacağım.

@kadirnar
Copy link
Contributor Author

@fcakyon @devrimcavusoglu
Local de tüm testlerden geçiyor. Zaten demo dosyasını da hazırladım hiçbir hata yok. Fakat github testlerinden geçemiyor maalesef. Buna benim yapabileceğim pek bir şey yok. İsterseniz kapatabilirsiniz.

Adding autoShape... 
10/11/2022 16:13:36 - INFO - utils.torch_utils -   YOLOR 🚀 2022-10-11 torch 1.11.0 CUDA:0 (NVIDIA GeForce GTX 1050 Ti, 4040.125MB)

.
----------------------------------------------------------------------
Ran 3 tests in 35.056s

OK

@fcakyon
Copy link
Collaborator

fcakyon commented Oct 11, 2022

@kadirnar load diyince load edemiyor, dedigin gibi kapatabiliriz, cok stabil olmuyor herhalde torch hubdan yuklemek

eline saglik yine de tum ugraslarin icin 💯

@kadirnar kadirnar closed this Oct 12, 2022
@JuanFKurucz
Copy link

Regarding this occurrence, is there a plan to get support for yolov7? We were looking forward to it

@kadirnar
Copy link
Contributor Author

@JuanFKurucz

Thank you for your interest. We closed it because it didn't pass Github tests. I'll rewrite when the pip version is released.
Yolov7-SAHI: https://github.com/kadirnar/Yolov7-SAHI

@fcakyon
Can you share this repo in the discussions section?

@kadirnar kadirnar deleted the yolov7 branch October 12, 2022 18:35
@fcakyon
Copy link
Collaborator

fcakyon commented Oct 19, 2022

@JuanFKurucz we are currently waiting for any yolov7 pip installable version to add the sahi yolov7 support. In the meantime, you can use the above source that @kadirnar linked :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants