search

buerokratt / Buerokratt-Chatbot

1 stars 19 forks source link

Get rid of [doc...] references in the LLM text (or replace with web urls) #728

Open PaulaMerle opened 5 months ago

PaulaMerle commented 5 months ago

AS A Enduser I WANT TO see clear responses SO THAT responses would be clearer to understand

GUI

Example

Screenshot 2024-05-27 at 11 39 20

Response: Abielu võib lõppeda mitmetel juhtudel, näiteks: 1. Abielulahutus: Abikaasad võivad otsustada lahutada ning esitada abielulahutuse avalduse kohtule . 2. Abikaasa surm: Kui üks abikaasadest sureb, lõpeb abielu automaatselt ja surnud abikaasa osa ühisvarast kuulub tema pärandvarasse [doc5]. 3. Abielu kehtivuse tunnistamine: Kui abielu on kehtetu, võib kohus tunnistada selle kehtetuks ja abielu lõpeb . 4. Abielu tühistamine: Kohus võib tühistada abielu, kui see on sõlmitud seadusevastaselt või mõnel muul alusel . 5. Abikaasade kokkulepe: Abikaasad võivad omavahel kokku leppida abielu lõpetamises ning selleks sõlmida abieluvaralepingu [doc4]. Loodan, et see vastus oli teile kasulik. Kui teil on veel küsimusi, siis palun andke mulle teada.

Should be: Abielu võib lõppeda mitmetel juhtudel, näiteks: 1. Abielulahutus: Abikaasad võivad otsustada lahutada ning esitada abielulahutuse avalduse kohtule. 2. Abikaasa surm: Kui üks abikaasadest sureb, lõpeb abielu automaatselt ja surnud abikaasa osa ühisvarast kuulub tema pärandvarasse. 3. Abielu kehtivuse tunnistamine: Kui abielu on kehtetu, võib kohus tunnistada selle kehtetuks ja abielu lõpeb. 4. Abielu tühistamine: Kohus võib tühistada abielu, kui see on sõlmitud seadusevastaselt või mõnel muul alusel. 5. Abikaasade kokkulepe: Abikaasad võivad omavahel kokku leppida abielu lõpetamises ning selleks sõlmida abieluvaralepingu. Loodan, et see vastus oli teile kasulik. Kui teil on veel küsimusi, siis palun andke mulle teada.

PaulaMerle commented 4 months ago

Tested on 11.07 in the test environment and none of ACs is fulfilled. Still both [docN] and empty space before "." at the end of sentence appear in the text. Image

ValterAro commented 3 months ago

Updated the regex as it was missing some of the cases. The new PR is here https://github.com/buerokratt/Buerokratt-Chatbot/pull/832 .

ValterAro commented 3 months ago

Seems to work as intended.