jhagara / TP-DeepSearch

Team 19 code base repository.
https://team19-16.studenti.fiit.stuba.sk
0 stars 1 forks source link

Origin/feature/path validation #120

Closed xsitarcik closed 7 years ago

xsitarcik commented 7 years ago

Added path validation feature

mateee12 commented 7 years ago

Ale uvedomil som si ze tie tvoje files budu vlastne standalone, tak prosimta vsetky veci co si tu povytvaral umiestni v priecinku helper do nejakeho priecinka specialneho - nazov je na tebe a skopci tam aj schemu ktoru robil jakub na to issue xml a zvaliduj aj to, dikes

mateee12 commented 7 years ago

Kukal som ako ma jakub vytvoreny ten schema validator, mali by to byt nakodene genericky, takze cely tento validator pouzi na validovanie issue xml podla jeho schemy + marc xml podla tvojej schemy, ale vsetky potrebne veci si teda nakopci do separatneho priecinka v priecinku helper, nech to je odseparovane

xsitarcik commented 7 years ago

Akosi nerozumiem. Ved uz vsetky potrebne veci su v separatnom priecinku a aj jakubovu schemu na validovanie issue xml pouzivam

mateee12 commented 7 years ago

Aha jasne, nevsimol som si dnes ze uz vcera si tam dal zmeny, ma to prekvapilo ze ako rychlo, super, tak ten predposledny koment moj ignoruj, dnes to spustim na windowse, ked to pojde tak to odklepnem, ale vypada to dobre :+1:

xsitarcik commented 7 years ago

Len take info ked to budes spustat, ze teraz s tou validacou xml issue to bezi niekolko desiatok nasobne pomalsie, kedze lxml z kazdeho issue parsuje ten etree, co trva take 2-3 sekundy, teda jeden rocnik s ~200 issue validuje ~10minut, predtym to bola otazka par sekund

mateee12 commented 7 years ago

nasiel som par veci co potrebuju opravit, ked to opravis tak to riadne rucne otestuj, jak som to robil ja teraz: