divvun / divvun-gramcheck-web

Grammar checker for web word processors, targeted at minority and indigenous languages, but open for everyone.
GNU General Public License v3.0
1 stars 0 forks source link

It seems grc versions are not updated on the server #71

Open Trondtr opened 2 years ago

Trondtr commented 2 years ago

For fao grc, the following sentence:

Hann skal læra at virða tær skyldur og tey rættindi, sum fólkaræði byggir á, har ímillum teir sáttmálar, ið Sameindu Tjóðir hava samtykt í sambandi við barnarættindi og mannarættindi annars.

gets a correction samtykt to samtykja, and the error message "Det ser ut til at verbet ditt ikkje skulle ende på ð.", corresponding to the tag &msyn-sup-inf. The problem is that this (false) alarm cannot be reproduced on the command line. The offending rule was removed from the grammarchecker.cg3 file on february 6th.

The conclusion is thus that MS Word accesses an online grammar version dating from before February 6th. I do not know whether this holds for fao only, or whether it goes for all the online grammars, but it should be looked into. The fao grc is in use by a whole language community (the largest of the ones we serve) and the issue should thus get some priority. If the error carries over to the other grc versions the issue is even worse. Due to bug #70, I have only tested on MS Word.

snomos commented 2 years ago

This bug is really about how and when to update the grammar checker *.zcheck files on the production server. We have a standardised process for rolling out new spellers, but nothing for grammar checkers.

Trondtr commented 2 years ago

While waiting, we should document how to inspect the dates for the grammarcheckers.

Trondtr commented 1 year ago

After update on the server (today), I now check as describet above: I add the following sentence

Hann skal læra at virða tær skyldur og tey rættindi, sum fólkaræði byggir á, har ímillum teir sáttmálar, ið Sameindu Tjóðir hava samtykt í sambandi við barnarættindi og mannarættindi annars.

and send it to checkng. Now (contrary to yesterday) the setence comes out as true negative. The update thus is done, and the new version works.

The bug should not be closed, though: What happened today was that 3 languages were manually updated (fo.zcheck, nb, smn). The other languages were not updated, and the automatic update procedure is also not in place. Also, the documentation for how to check for dates (either via easter eggs or via inspecting the dates on pahkat.uit.no) is not in place.

The good news is that 3 languages are updated, so that the last 9 months of work is available for the language communities. Thanks!

Trondtr commented 1 year ago

This bug is really about how and when to update the grammar checker *.zcheck files on the production server. We have a standardised process for rolling out new spellers, but nothing for grammar checkers.

So, let us partition the problem: if I understand correctly, we need two things:

  1. a operational process for ensuring regression.
  2. add the *.zcheck file to the list of files to be copied automatically to the server

There should be no reason for waiting months for the second point, it seems like a five-minute-fix, at most. The issue thus seems to have stranded at the first point.

I do not whether speller update is linked to the make check procedure for transducers, but if it is, it should be straightforward to to the same for grammar checkers. We do have a test procedure, and it can be used for this purpose as well.