obss
add YOLOX model support
SAHI kütüphanesine YOLOX modelini en basit haliyle ekledim. #535
Düzeltilecekler:
- [X] Resize modulu yazıldı.
- [ ] Test kodu yazılacak.
- [ ] Notebook dosyası oluşturulacak.
Resize kodunu özellikle yazmadım. SAHI için ortak bir resize modulu oluşturabilir miyiz? Yolox reposundaki örneği incelleyip geri dönüş yapabilir misiniz? YOLOX-RESIZE
Ayrıca nms_thre değerini kullanıcıdan istesek nasıl olur? Şuan ben manuel değer veriyorum. Aynı durum yolov7 modeli içinde geçerli.
@kadirnar resize islemi her kutuphaneye ozel oluyor, yolov5/v7de kendi icinde hallediyor mesela, o yuzden resize isleminin frameworke ozel olarak hazirlanmasi gerekiyor.
nms icin ek input almaya gerek yok, yolox kutuphanesindeki default deger (0.3) kullanilabilir, isteyen extra DetectionModel.model.nmsthre =0.4 seklinde de set edebilir (self.model e Predictor instanceini atiyorsun diye kabul ediyorum)
anladigim kadariyla yolox kutuphanesi icin ekstra resize islemi yapmaya gerek yok, resizei kendi icinde hallediyor: https://github.com/Megvii-BaseDetection/YOLOX/blob/74b637b494ad6a968c8bc8afec5ccdd7ca6b544f/tools/demo.py#L140-%20L146
image_size degerini test_size olarak set etmemiz yeterli olur gibi anliyorum: https://github.com/Megvii-BaseDetection/YOLOX/blob/74b637b494ad6a968c8bc8afec5ccdd7ca6b544f/tools/demo.py#L118
nms icin ek input almaya gerek yok, yolox kutuphanesindeki default deger (0.3) kullanilabilir, isteyen extra DetectionModel.model.nmsthre =0.4 seklinde de set edebilir (self.model e Predictor instanceini atiyorsun diye kabul ediyorum)
Hubconf içinde Predictor kullanılıyor mu bilmiyorum. Yolox içindeki örneği alıp bir fonksiyon olarak kodlamayı düşünüyorum. Veya hubconf bu satırdaki exp classında img_size değeri manuel alınmış.
exp.input_size= input_size
Parametresini ekleyip fonksiyonlardan da bunu istersek daha iyi olabilir.
@kadirnar source baktim, yolox projesindeki hubconfta size hardcoded set edilmis.
bence ugrasmaya gerek yok, kullanici yolox icin 640 disinda bir size set etmeye calisirsa yoloxdetectionmodel image_size set etmeyi desteklemiyor gibi bir warning vermemiz yeterli olur 👍
@kadirnar source baktim, yolox projesindeki hubconfta size hardcoded set edilmis.
bence ugrasmaya gerek yok, kullanici yolox icin 640 disinda bir size set etmeye calisirsa yoloxdetectionmodel image_size set etmeyi desteklemiyor gibi bir warning vermemiz yeterli olur 👍
YOLOX reposundaki resize modülünü copy-paste yaptım. Boxes kayma hataları da giderildi. Tüm resize değerlerini destekliyor. Müsait zamanda test ve notebook dosyalarını oluşturacağım👍
@kadirnar firsat buldugunda main branchteki yenilikleri rebase ya da merge ile alip devam edebilirsin calismalarina
@kadirnar firsat buldugunda main branchteki yenilikleri rebase ya da merge ile alip devam edebilirsin calismalarina
Şuan bu repo sadece local de çalışacak şekilde uygun(Config dosyası full path istiyor). Config parametrelerini farklı bir modül ile alınması gerekiyor. Belki pip ile ana repodan farklı bir detect kodu ile çözülebilir. Biraz karışık geldiği için askıya aldım.
@fcakyon YOLOX ve YOLOV7 modellerini ayrı ayrı oluşturmak yerine https://github.com/open-mmlab/mmyolo direk bu kütüphaneyi ekleyebiliriz.
@fcakyon YOLOX ve YOLOV7 modellerini ayrı ayrı oluşturmak yerine https://github.com/open-mmlab/mmyolo direk bu kütüphaneyi ekleyebiliriz.
evet mantikli, yolov7 destegi eklemeyi dusunuyolar mi diye issue acilabilir o repoya 👍
Can we use sahi with other object detection models not mentioned in the repository as well?? Sahi'yi depoda belirtilmeyen diğer nesne algılama modelleriyle de kullanabilir miyiz? for eg SSD, yolov4/5?
@Rushikesh-pawar yes you can definitely use. You have to add the support yourself. You can follow the steps in main readme under contributing section 👍
@fcakyon Model dosyası güncelledikten sonra yeni PR açacağım. Yolov7 ve YOLOX PR'ları şimdilik kapatıyorum.
could anyone explain it in English ?
https://github.com/Resham-Sundar/sahi-yolox