Skip to content

add torchvision object detector support - #368

Closed
kadirnar wants to merge 80 commits into
obss:mainfrom
kadirnar:torchvision
Closed

kadirnar wants to merge 80 commits into
obss:mainfrom
kadirnar:torchvision

Conversation

@kadirnar

@kadirnar kadirnar commented Jan 28, 2022 •

Copy link
Copy Markdown
Contributor

-model.eval eklendi
-kütüphane hataları düzeltildi.
-mask hatası yorum satırı yapıldı

test_torchvision.py:
-fonksiyonların altına kütüphaneler eklendi
-download_torchvision_model eklendi
-test dosyasındaki box sayıları düzeltilemedi.
Comment thread sahi/model.py Outdated
Comment thread sahi/model.py Outdated
Comment thread sahi/model.py Outdated
Comment thread sahi/model.py Outdated
Comment thread sahi/utils/torchvision.py Outdated
Comment thread tests/test_torchvision.py Outdated
Comment thread tests/test_torchvision.py Outdated
Comment thread sahi/utils/torchvision.py Outdated
-classes yerine COCO_CLASSES olarak düzeltildi.
-category_remapping fonksiyonu yazılarak category id değişkeni yazıldı
-mask hatası çözülemedi.

test_torchvision.py:
-assert kodları daha sonra eklenmek üzere kaldırıldı

torchvision.py
- classes yerine COCO_CLASSES ismi tercih edildi.
@kadirnar

kadirnar commented Jan 29, 2022 •

Copy link
Copy Markdown
Contributor Author

https://github.com/kadirnar/sahi/blob/torchvision/sahi/model.py#L786-L789

Buradaki kodların yazılması hakkında yardıma ihtiyacım var. Şimdilik mask işlemin yapılmasını istemiyorum. Object detection kodlarının başarılı şekilde çalışıp çalışmadığını kontrol etmek istiyorum.

@kadirnar

kadirnar commented Feb 6, 2022 •

Copy link
Copy Markdown
Contributor Author

@fcakyon SAHI algoritmasına TorchVision kütüphanesi başarılı şekilde eklenmiştir. https://github.com/obss/sahi/issues/275 sorusuna bu branch'i yönlendirebilirsiniz.

Bundan sonra yapılacaklar:

  • test_torchvision.py dosyasına koordinat bilgileri eklenip tekrardan test edilecek.
  • Veri dönüşüm işlemleri için test kodu yazılacak.
  • Demo için notebook oluşturulacak.
  • Test ve model dosyasındaki kodlar daha basit hale getirilecek.

@kadirnar
kadirnar requested a review from fcakyon February 6, 2022 11:27
- model.py dosyasındaki gereksiz kodlar silindi.
- test_torchvision.py dosyasında etiket bilgileri girerek test kodları yazıldı.
- model.py dosyasındaki çakışan kod hatası düzeltildi.
- resim yolları github'a göre ayarlandı.
- reformat kodları çalıştırıldı.
@fcakyon fcakyon self-assigned this Feb 12, 2022
@fcakyon

fcakyon commented May 28, 2022

Copy link
Copy Markdown
Collaborator

@kadirnar resize sorununu cozdum su comitten nasil cozdugume bakabilirsin: f0319c4

son halini pullayip, configden model yukleme tarafina baslayabilirsin 👍

@devrimcavusoglu devrimcavusoglu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Throughout all of the files typo correction is required. Correct the typo in naming from FASTERCNN -> FASTERRCNN

@fcakyon fcakyon changed the title add torchvision support add torchvision object detector support Jun 16, 2022
@fcakyon

fcakyon commented Jun 16, 2022

Copy link
Copy Markdown
Collaborator

Closing this PR in favor of #486.

@fcakyon fcakyon closed this Jun 16, 2022
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.

Question: plans for generic TorchVision models?

3 participants