bito-code-review[bot] commented on code in PR #41660: URL: https://github.com/apache/superset/pull/41660#discussion_r3510252739
########## superset/translations/pl/LC_MESSAGES/messages.po: ########## @@ -2903,27 +3104,45 @@ msgstr "Nie można usunąć bazy danych, do której są przypisane zestawy danyc msgid "Cannot delete system themes" msgstr "Nie można uzyskać dostępu do zapytania" Review Comment: <div> <div id="suggestion"> <div id="issue"><b>Wrong translation for msgid</b></div> <div id="fix"> The `msgstr` for `"Cannot delete system themes"` is cross-contaminated — it reads "Nie można uzyskać dostępu do zapytania", which is the correct translation of the *different* entry `"Cannot access the query"` at line 3098. The `backfill_po.py` script pre-populates `msgstr` from the existing (stale, `#, fuzzy`) entry before calling the LLM, and the model apparently hallucinated/confabulated the translation rather than producing a new one. This would cause a wrong message to be displayed to Polish users when system themes cannot be deleted. </div> </div> <small><i>Code Review Run #437a94</i></small> </div> --- Should Bito avoid suggestions like this for future reviews? (<a href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>) - [ ] Yes, avoid them ########## superset/translations/pl/LC_MESSAGES/messages.po: ########## @@ -7140,15 +7639,21 @@ msgstr "Filtry według miar" msgid "Filters for comparison must have a value" msgstr "Filtry porównawcze muszą mieć wartość" +# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: ca, cs, de, +# es, fr, ja, lv, mi, ro, ru, sk, sr, sr_Latn, tr, uk] +#, fuzzy msgid "Filters for values equal to this exact value." -msgstr "" +msgstr "Filtruje wartości równe tej dokładnej wartości." #, fuzzy msgid "Filters for values greater than or equal." Review Comment: <div> <div id="suggestion"> <div id="issue"><b>Mismatched msgid-msgstr pair</b></div> <div id="fix"> Line 7650 has a completely unrelated msgstr: the English "Filters for values greater than or equal." is paired with the Polish translation of a different msgid ("`row_limit` musi być większe lub równe 0", from line 19514). This appears to be a leftover stale fuzzy entry from a prior pass — the script correctly detected it was fuzzy and replaced the empty msgstr, but the replacement itself is wrong. Review and correct the translation, then remove the `#, fuzzy` flag when verified. </div> </div> <small><i>Code Review Run #437a94</i></small> </div> --- Should Bito avoid suggestions like this for future reviews? (<a href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>) - [ ] Yes, avoid them ########## superset/translations/pl/LC_MESSAGES/messages.po: ########## @@ -4066,14 +4356,20 @@ msgstr "SQL skopiowano!" msgid "Copy" msgstr "Copier" +# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: ca, cs, de, +# es, fr, ja, lv, mi, pt_BR, ro, ru, sk, sr, sr_Latn, tr, uk] +#, fuzzy msgid "Copy SELECT statement" -msgstr "" +msgstr "Skopiuj instrukcję SELECT" msgid "Copy SELECT statement to the clipboard" msgstr "Spokiuj wynik SELECT do schowka" Review Comment: <div> <div id="suggestion"> <div id="issue"><b>Garbled pre-existing translation</b></div> <div id="fix"> Pre-existing entry "Copy SELECT statement to the clipboard" is badly garbled ("Spokiuj wynik SELECT do schowka" — 'Spokiuj' is not a Polish verb; correct would be "Skopiuj instrukcję SELECT do schowka"). This predates the backfill_po.py diff but was found during this review of the same diff region. </div> </div> <small><i>Code Review Run #437a94</i></small> </div> --- Should Bito avoid suggestions like this for future reviews? (<a href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>) - [ ] Yes, avoid them ########## superset/translations/pl/LC_MESSAGES/messages.po: ########## @@ -4066,14 +4356,20 @@ msgstr "SQL skopiowano!" msgid "Copy" msgstr "Copier" Review Comment: <div> <div id="suggestion"> <div id="issue"><b>Wrong-language pre-existing translation</b></div> <div id="fix"> Pre-existing entry "Copy" → "Copier" is a French translation that likely entered via an automated pipeline mixing languages. Replace with Polish "Kopiuj". This predates the backfill_po.py diff but sits in the same diff hunk. Note: the two backfill entries ("Copy SELECT statement", "Copy URL") added by this PR are correctly translated. </div> </div> <small><i>Code Review Run #437a94</i></small> </div> --- Should Bito avoid suggestions like this for future reviews? (<a href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>) - [ ] Yes, avoid them ########## superset/translations/pl/LC_MESSAGES/messages.po: ########## @@ -8995,11 +9630,17 @@ msgstr "Maksymalny promień" msgid "Maximum Width" msgstr "Minimalna szerokość" Review Comment: <div> <div id="suggestion"> <div id="issue"><b>Semantically inverted translation</b></div> <div id="fix"> `msgid "Maximum Width"` is translated as `"Minimalna szerokość"` — the word for 'Minimum Width' in Polish. `max` and `min` are semantically opposite; users will see a label that contradicts the actual feature limit, leading to confusion. </div> </div> <small><i>Code Review Run #437a94</i></small> </div> --- Should Bito avoid suggestions like this for future reviews? (<a href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>) - [ ] Yes, avoid them ########## superset/translations/pl/LC_MESSAGES/messages.po: ########## @@ -16389,19 +17576,28 @@ msgstr "" msgid "Unable to create chart without a query id." msgstr "Nie można utworzyć wykresu bez identyfikatora zapytania." +# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: cs, de, ja, +# lv, ru, sk, sr, sr_Latn, tr, uk] +#, fuzzy msgid "" "Unable to create report: User email address is required but not found. " "Please ensure your user profile has a valid email address." msgstr "" +"Nie można utworzyć raportu: adres e-mail użytkownika jest wymagany, ale " +"nie został znaleziony. Upewnij się, że Twój profil użytkownika ma " +"prawidłowy adres e-mail." Review Comment: <div> <div id="suggestion"> <div id="issue"><b>Machine-translation review incomplete</b></div> <div id="fix"> Machine-translated entry for 'Unable to create report' added at lines 17579-17588. Human review required per howtos.md before fuzzy flag removal. </div> </div> <small><i>Code Review Run #437a94</i></small> </div> --- Should Bito avoid suggestions like this for future reviews? (<a href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>) - [ ] Yes, avoid them ########## superset/translations/pl/LC_MESSAGES/messages.po: ########## @@ -16411,10 +17607,16 @@ msgstr "Nie można znaleźć takiego święta: [%(holiday)s]" msgid "Unable to generate download payload" msgstr "Nie można załadować pulpitu nawigacyjnego" +# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: cs, de, fr, +# ja, lv, ro, ru, sk, sr, sr_Latn, tr, uk] +#, fuzzy msgid "" "Unable to identify temporal column for date range time comparison.Please " "ensure your dataset has a properly configured time column." msgstr "" +"Nie można zidentyfikować kolumny czasowej do porównania zakresu dat. " +"Upewnij się, że Twój zestaw danych ma prawidłowo skonfigurowaną kolumnę " +"czasu." Review Comment: <div> <div id="suggestion"> <div id="issue"><b>Machine-translation review incomplete</b></div> <div id="fix"> Machine-translated entry for 'Unable to identify temporal column' added at lines 17610-17619. Human review required per howtos.md before fuzzy flag removal. </div> </div> <small><i>Code Review Run #437a94</i></small> </div> --- Should Bito avoid suggestions like this for future reviews? (<a href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>) - [ ] Yes, avoid them ########## superset/translations/pl/LC_MESSAGES/messages.po: ########## @@ -14086,10 +15043,15 @@ msgstr "Nie udało się utworzyć etykiety." msgid "Task could not be updated." msgstr "Nie udało się zaktualizować etykiety." +# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: de, sr, +# sr_Latn] +#, fuzzy msgid "" "Task is not abortable. The task is in progress but has not registered an " "abort handler." msgstr "" +"Zadanie nie może zostać przerwane. Zadanie jest w toku, ale nie " +"zarejestrowano procedury obsługi przerwania." #, fuzzy msgid "Task parameters are invalid." Review Comment: <div> <div id="suggestion"> <div id="issue"><b>Semantic mistranslation of Task parameters</b></div> <div id="fix"> Pre-existing fuzzy entry near the new backfill additions (lines 15064-16118): `msgid "Task parameters are invalid."` is translated as "Parametry etykiety są nieprawidłowe." — 'etykiety' (label's) is wrong; the message is about task/background-job parameters, not label parameters. Polish: "Parametry zadania są nieprawidłowe." </div> </div> <small><i>Code Review Run #437a94</i></small> </div> --- Should Bito avoid suggestions like this for future reviews? (<a href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>) - [ ] Yes, avoid them ########## superset/translations/pl/LC_MESSAGES/messages.po: ########## @@ -16322,18 +17500,27 @@ msgstr "Typ porównania: różnica wartości lub procent" msgid "Type to search (contains)..." msgstr "Szukaj w kolumnach" +# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: de, sr, +# sr_Latn] +#, fuzzy msgid "Type to search (ends with)..." -msgstr "" +msgstr "Wpisz, aby wyszukać (kończy się na)..." +# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: de, sr, +# sr_Latn] +#, fuzzy msgid "Type to search (starts with)..." -msgstr "" +msgstr "Wpisz, aby wyszukać (zaczyna się od)..." Review Comment: <div> <div id="suggestion"> <div id="issue"><b>Machine-translation review incomplete</b></div> <div id="fix"> This block adds 2 machine-translated entries from `backfill_po.py` marked `#, fuzzy`. Per the review workflow in `howtos.md` (lines 403-408), each entry needs verification that the translation is contextually correct before the fuzzy flag is removed. No reviewer sign-off or evidence of human review is present in this diff. </div> </div> <small><i>Code Review Run #437a94</i></small> </div> --- Should Bito avoid suggestions like this for future reviews? (<a href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>) - [ ] Yes, avoid them ########## superset/translations/pl/LC_MESSAGES/messages.po: ########## @@ -14099,18 +15061,28 @@ msgstr "Parametry etykiety są nieprawidłowe." msgid "Tasks" Review Comment: <div> <div id="suggestion"> <div id="issue"><b>Semantic mistranslation of Tasks</b></div> <div id="fix"> Pre-existing fuzzy entry near a new backfill addition (same diff hunk lines 15064-16118): `msgid "Tasks"` translates to "tagi" (tags), which is incorrect — "Tasks" refers to a background-job or task-management UI panel, not tags. The correct Polish is "Zadania". </div> </div> <small><i>Code Review Run #437a94</i></small> </div> --- Should Bito avoid suggestions like this for future reviews? (<a href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>) - [ ] Yes, avoid them ########## superset/translations/pl/LC_MESSAGES/messages.po: ########## @@ -16449,8 +17651,11 @@ msgstr "" "ponownie później. Skontaktuj się z administratorem, jeśli problem będzie " "się powtarzał." +# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: ca, cs, de, +# es, fa, fr, ja, lv, mi, ro, ru, sk, sr, sr_Latn, tr, uk] +#, fuzzy msgid "Unable to parse SQL" -msgstr "" +msgstr "Nie można przeanalizować SQL" Review Comment: <div> <div id="suggestion"> <div id="issue"><b>Machine-translation review incomplete</b></div> <div id="fix"> Machine-translated entry for 'Unable to parse SQL' added at lines 17654-17658. Human review required per howtos.md before fuzzy flag removal. </div> </div> <small><i>Code Review Run #437a94</i></small> </div> --- Should Bito avoid suggestions like this for future reviews? (<a href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>) - [ ] Yes, avoid them ########## superset/translations/pl/LC_MESSAGES/messages.po: ########## @@ -18702,10 +20013,15 @@ msgstr "nieprawidłowy e-mail" msgid "is" Review Comment: <div> <div id="suggestion"> <div id="issue"><b>Incorrect auxiliary verb translation</b></div> <div id="fix"> The msgid `"is"` is the auxiliary verb 'to be' (3rd person singular), not the plural noun 'baskets'. In Polish, this should translate to `"jest"` (he/she/it is). The current translation `"Kosze"` means 'baskets' and is semantically incorrect. Cross-referenced with German translation which correctly uses `ist` for the same msgid. </div> </div> <small><i>Code Review Run #437a94</i></small> </div> --- Should Bito avoid suggestions like this for future reviews? (<a href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>) - [ ] Yes, avoid them ########## superset/translations/pl/LC_MESSAGES/messages.po: ########## @@ -16459,8 +17664,11 @@ msgstr "Pobieranie obrazu nie powiodło się, odśwież i spróbuj ponownie." msgid "Unable to retrieve dashboard colors" msgstr "Nie można pobrać kolorów pulpitu nawigacyjnego" +# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: ca, cs, de, +# es, fr, ja, lv, mi, ro, ru, sk, sr, sr_Latn, tr, uk] +#, fuzzy msgid "Unable to sync permissions for this database connection." -msgstr "" +msgstr "Nie można zsynchronizować uprawnień dla tego połączenia z bazą danych." Review Comment: <div> <div id="suggestion"> <div id="issue"><b>Machine-translation review incomplete</b></div> <div id="fix"> Machine-translated entry for 'Unable to sync permissions' added at lines 17667-17671. Human review required per howtos.md before fuzzy flag removal. </div> </div> <small><i>Code Review Run #437a94</i></small> </div> --- Should Bito avoid suggestions like this for future reviews? (<a href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>) - [ ] Yes, avoid them -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
