search

derlem / kanarya

A deep learning model for classification of 'de/da' clitics in Turkish
21 stars 3 forks source link

WIP: Merge the tiny dockerization work into master #33

Open onurgu opened 4 years ago

haozturk commented 4 years ago

Requirement dosyaları değiştiği için webapp'i çalıştırmak için gereken adımları anlatan şu README'yi de update etmemiz gerekir mi sence? @onurgu

uskudarli commented 4 years ago

Gerekiyor olmasi @haozturk.

Su anda testleri yaparken bastan sona temiz kurulum ile ilerlemeli. Aradaki tum puruzleri de gidermeye calisalim. Model'i open access yapacagiz. O da yine bir degisiklik yaratacak README de.

haozturk commented 4 years ago

Evet hocam, modeli open access yaptıktan sonra başka bir PR'da README'yi update'leriz. @uskudarli

ulgens commented 4 years ago

@onurgu Selam, phase 1 - phase2 durumu yerine poetry ya da pipenv gibi bi alet daha iyi sonuç verebilir gibi orada. Okay dersek, docker configini ona göre güncelleyebilirim.

onurgu commented 4 years ago

Requirement dosyaları değiştiği için webapp'i çalıştırmak için gereken adımları anlatan şu README'yi de update etmemiz gerekir mi sence? @onurgu

evet gerekir, bu PR WIP olsun zaten. her şey uyumlu olunca merge ederiz

herkese yorumları için teşekkürler. bir daha ihtiyaç duyduğumda haberdar ederim