add Tensorflow Hub detector support

[kadirnar]

Testlerden geçebilmesi için tensorflow ve tensorflow_hub kütüphanelerin kurulması gerekiyor.

• package_testing.yml
• ci.yml

- name: Install tensorflow and tensorflow_hub
  run: >
    pip install  tensorflow
    pip install tensorflow_hub

Bu format uygun mudur? @fcakyon

[fcakyon]

@kadirnar Su sekilde versiyon belirtebiliriz:

- name: Install tensorflow(2.9.1) and tensorflow_hub(0.12.0)
  run: >
    pip install  tensorflow==2.9.1
    pip install tensorflow_hub==0.12.0

[fcakyon]

vazgectim bu degisiklikten.

[kadirnar]

@fcakyon test kodları paket kurulumlarından geçemiyor. Bunun için nasıl düzeltme yapmalıyım? (Bu düzeltmeden sonra reformat yapacağım)

[fcakyon]

@fcakyon test kodları paket kurulumlarından geçemiyor. Bunun için nasıl düzeltme yapmalıyım? (Bu düzeltmeden sonra reformat yapacağım)

Testlerin calismasi icin black ve isort ile reformat yapip pushlaman gerekiyor. Linting'de takilmis tum testler.

[fcakyon]

@kadirnar su doctaki bilgiyi kullanarak tensorflow icin otomatik device secme eklememiz gerekiyor bu PRi mergeleyebilmemiz icin: https://www.tensorflow.org/guide/gpu#overview

ONEMLI NOT: maindeki son degisiklikleri mergeldim, localinde calismaya baslamadan once pullasan cok iyi olur 👍

[kadirnar]

@kadirnar su doctaki bilgiyi kullanarak tensorflow icin otomatik device secme eklememiz gerekiyor bu PRi mergeleyebilmemiz icin: https://www.tensorflow.org/guide/gpu#overview

ONEMLI NOT: maindeki son degisiklikleri mergeldim, localinde calismaya baslamadan once pullasan cok iyi olur 👍

Auto device kodlarını nasıl entegre etmemi tavsiye edersiniz? Tensorflowhub classı ıcındekı load model fonksiyonuna mı?

Yoksa from sahi.utils.tensorflow import is_tensorflow_cuda_available fonksiyonu mu yazmalıyım?

Örnek Kod:

if tf.test.is_gpu_available():
    self.device = "/gpu:0"
else:
    self.device = "/cpu:0"

Ayrıca test kodlarını çalıştırırken torch kütüphanesini de yüklemek zorunda oluyorum. (SAHI torch kütüphanesi ile yazıldığı için normal olduğunu düşündüm)

https://github.com/obss/sahi/blob/main/sahi/model.py#L668

[fcakyon]

Evet şu anda torch SAHI’de zorunlu dependency.

device olayı için şöyle bir şey yapabiliriz:
DetectionModel için set_device diye bir fonksiyon tanımlayıp şu anki device set etme satırını ona taşırız.

tensofflubhubdetectionmodel içinde set_device fonksiyonunu yeniden tanımlayabilirsin yazdığın şekilde

[kadirnar]

Evet şu anda torch SAHI’de zorunlu dependency.

device olayı için şöyle bir şey yapabiliriz: DetectionModel için set_device diye bir fonksiyon tanımlayıp şu anki device set etme satırını ona taşırız.

tensofflubhubdetectionmodel içinde set_device fonksiyonunu yeniden tanımlayabilirsin yazdığın şekilde

def set_device(self):
    import tensorflow as tf
    if self.device is None:
        try:
            if tf.test.is_gpu_available():
                self.device = "/gpu:0"
            else:
                self.device = "/cpu:0"
                
        except Exception as e:
            raise Exception("Error setting device: ", e)

Hata mesajı ve kodlar için öneriniz var mı? Yoksa bu şekilde göndereceğim.

[fcakyon]

Güzel olmuş eline sağlık, try except’e gerek olmayabilir. torch’ta yaptığımız gibi try except yapmadan set edebiliriz gpu/cpu

[kadirnar]

Readme.md dosyasındaki reformatter kodu hata veriyor. Eski reformatter kodunu çalıştırdım.

[fcakyon]

67-69 satirlari arasinin silinmesi, onun yerine set_device fonksiyonu cagirilarak device ayarlanmasi gerekiyor

[kadirnar]

SAHI + TensorFlowHub: https://github.com/kadirnar/Yolov7-SAHI/blob/main/demo/inference_for_tensorflowhub.ipynb