mi2-warsaw / CzasDojazdu

https://hub.docker.com/r/marcinkosinski/czasdojazdu/builds/
5 stars 1 forks source link

Problem z datą w danych - dlatego nie mamy danych na serwerze #14

Closed MarcinKosinski closed 8 years ago

MarcinKosinski commented 8 years ago

Michał czy mógłbyś opisać problem, tak żeby wszyscy wiedzieli @michalcisek

CC @mikolajjj @abrodecka

michalcisek commented 8 years ago

Nie mamy aktualnych ofert, ponieważ do bazy danych nie jest wgrywana data dodania ogłoszenia. Ręcznie jak przechodzimy przez skrypt to poprawnie jest scrapowana ta zmienna. Być może problem leży w formacie. Sprawdzę i dam znać.

michalcisek commented 8 years ago

Okej, problem jest z polskimi znakami występującymi w nazwie miesięcy. Przez to przy konwersji na datę, zwraca brak danych

MarcinKosinski commented 8 years ago

Mamy pomysł jak to naprawić?

W dniu 29 kwietnia 2016 18:27 użytkownik Michał Cisek < notifications@github.com> napisał:

Okej, problem jest z polskimi znakami występującymi w nazwie miesięcy. Przez to przy konwersji na datę, zwraca brak danych

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/mi2-warsaw/CzasDojazdu/issues/14#issuecomment-215793985

michalcisek commented 8 years ago

Tak, poprawię to.

MarcinKosinski commented 8 years ago

@mikolajjj - dlatego nie mieliśmy żadnej oferty w danych do prezentacji, bo zadna nie miala daty.

MarcinKosinski commented 8 years ago

@michalcisek sprawdzalem pobieranie danych dla gumtree : https://gist.github.com/MarcinKosinski/d197195d849977d9d16eb3ec68425fb9

to data pobiera sie okej (nie ma polski znakow w datach, bo data jest numeryczna). Czy moglbys jakos lepiej zobrazowac problem, bo nie widze gdzie ewentualnie szukac, zeby cos pomoc?

michalcisek commented 8 years ago

@MarcinKosinski problem jest dla olx-a. Tam data pobierana jest przykładowo z wyrażenia "Dodane o 17:12, 16 kwietnia 2016". Po obróbce, "kwietnia" jest zamieniane na "kwiecień", które zawiera polski znak i rodzi się problem. Trzeba po prostu zmienić, żeby podstawiał liczbę i będzie po kłopocie.

michalcisek commented 8 years ago

Poprawiłem, powinno być okej.