acikyazilimagi / depremadres-intent-classification-v0

37 stars 26 forks source link

Adding training notebook. #41

Closed devrimcavusoglu closed 1 year ago

devrimcavusoglu commented 1 year ago
devrimcavusoglu commented 1 year ago

hocam inceledim kodu,

deprem_intent_classification.ipynb'daki, normalize_text metotunun son satirindaki text.lower().strip() isleminde text=text.lower().strip() olmadigi icin son operasyon text uzerinde uygulanmamis oluyor, istersen bi goz atabilirsin sen de :)

Yanlış anlamadıysam son satırdan kastınız return text.lower().strip() değil mi ? Burada problem görünmüyor hocam, method chaining ile oluşan str object return ediliyor.

ktoprakucar commented 1 year ago

hocam inceledim kodu, deprem_intent_classification.ipynb'daki, normalize_text metotunun son satirindaki text.lower().strip() isleminde text=text.lower().strip() olmadigi icin son operasyon text uzerinde uygulanmamis oluyor, istersen bi goz atabilirsin sen de :)

Yanlış anlamadıysam son satırdan kastınız return text.lower().strip() değil mi ? Burada problem görünmüyor hocam, method chaining ile oluşan str object return ediliyor.

hocam simdi tekrar inceledim, ben sadece print kismina gore yorum yapmistim, asagida def __getitem__(self, idx): kisminda return edilen degeri kullanmissin, kusra bakma yanlis yorum yaptim :)