jhagara / TP-DeepSearch

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

bug assembler - strana s velkym separatorom #70

Open jhagara opened 7 years ago

jhagara commented 7 years ago

na vstupe je takato strana strana14_vyplnena na obrazku je znazornene ocakavane prepojenie. fulltext v pravo hore sa ale pripoji k prvemu clanku(cervena), mal by byt v clanku zelena

jhagara commented 7 years ago

tu je postup ako priraduje algoritmus ten fulltext najprv najde samostatny fulltext v strede strana14_postup2 potom najde najblizsiu skupinu nad nou co je separator strana14_postup3 potom v cykle hlada taky ktory je najblizsie nad nim a nie je separator strana14_postup4 najde fulltext v vlavo - hlada od najlavejsieho. tento fulltext ale patri k inemu clanku.

jhagara commented 7 years ago

problem je pri hladani najblizsej groupy pre aktualnu groupu v __find_nearest_above. mozne riesenie je nehladat podla celej sirky groupy ale iba podla stredu groupy. skusal som 50% stred a vtedy to zbehlo. len neviem ci to nepokazi nieco ine.

jhagara commented 7 years ago

rozdelit separator

jhagara commented 7 years ago

rozdelil som ten separator a uz ten clanok dava dobre. teraz je ale problem s tym clankom dole. strana14_2diii2

Pozije sa pravidlo 2diii2 , ktore priradi text v pravo k strednemu nesamostatnemu textu, teda k clanku v strede.

Myslim ze by sa dalo doplnit to pravidlo takto:

Ak je samostatny text v pravo a nad nim je separator a pod nim nie je nic (je v pravo dole) tak ho prirad k textu v lavo.

Nemalo by to pokazit ostatne pravidla kedze je to specifikovane ze musi mat nad sebou separator a pod sebou nic. A kedze ma nad sebou separator tak by pravdepodobne nemal ist niekde vysie ako je, teda by mal ist dolava.

mateee12 commented 7 years ago

Noo, len musel by si teda vlozit novu logiku pre to co si hovoril, teda: "Ak je samostatny text v pravo a nad nim je separator a pod nim nie je nic (je v pravo dole) tak ho prirad k textu v lavo." A musel by si aj rozdelit separator na dve casti a to neviem ako vies kedy ho rozdelit na dve casti a kedy nie

jhagara commented 7 years ago

Ta logika je uz implementovana, aj odskusana na mojom teste :D . Separator podla toho co viem by pri urcovani separatorov mal byt rozdeleny. A aj keby nebol tak mna zaujima ci je priamo nad nim separator, aky je velky je nepodstatne.

mateee12 commented 7 years ago

No sak ale ako potom docielis aby pravy horny fulltext sa priradil spravne do tej zelenej casti?

mateee12 commented 7 years ago

To ma zmiatlo, lebo ty mas nakresleny jeden separator, realne toto vlastne nebudes mat, ale budes tam mat 4 separatory presne podla tych stlpcov

mateee12 commented 7 years ago

No teda potom ak das to ze ak je osamoteny stlpce v pracom stlpci a pod nim uz nieje nic a hned nad nim je separator tak ho prirad k lavemu, toto by teda mohlo byt oka, len treba ceknut predosle vysledkz ci to nejako nepokazi uspesnost + testy musia zbehnut. Dorobis to este tento sprint?

jhagara commented 7 years ago

ano budu tam separatory pre kazdy stlpec. v mojom povodnom teste bol spolu ale potom som ho rozdelil. mal som to napisat jasnejsie. ako som uz pisal implementovane to je. pozeral som to na par cisiel (asi 5) toho slovaka ci to nepokazi zvysok pravidiel a malo by to byt v pohode. a skusal som to na mojom teste kde to zbehlo. mozem to potom este odskusat tak, ze to aj realne spustim na par cislach.

jhagara commented 7 years ago

otestoval som to, ale nezda sa ze by to pomohlo. dal som to pravidlo zatial prec a treba sa na to este pozriet, ale asi az dalsi sprint. test teda stale nezbehne.