keeleinstituut / tv-tolkevarav

Tõlkevärav (Translation Hub)
1 stars 0 forks source link

CAT - splitting xliff pieces #727

Open plakitkelly opened 3 months ago

plakitkelly commented 3 months ago

Wiki 7.2b.3 Actual result: there are options from 2 to 100. If selecting 51-100, user sees error: The chunks count field must be between 2 and 50. Expected result: According to wiki, there should be dropdown option from 2 up to 50. Remove options 51-100 and empty option then

To-do:

MariusJulius commented 2 months ago

Matecat logic related problem. We can remove option over 50, but this doesn't guarantee splitting always works as it is related to matecat logic - you cannot always split to desired amounts (related to splitting logic) - eg text is short and user wants to have too many pieces.

thenouan commented 2 months ago

@MariusJulius issue is with selection, for options 51-100 an error is guaranteed since it is not supported to split it into that many chunks; for options 1-50 error depends on the splittable content, which is acceptable

Issue is not in matecat, but in FE service, which displays these options

MariusJulius commented 3 weeks ago

As analyses is in progress (hardware limitations) it might not be possible to test this.

MariusJulius commented 3 weeks ago

I tested one of the older orders which had analyses. Splitting and merging worked. I could split up to 50.

Only thing I noticed was that when splitting to 2 and then again to 2 it created 1 additional chunk so 3 in total. But I suppose this is a corner case. You can always merge to one and have the number of chunks needed.