You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by "rusackas (via GitHub)" <gi...@apache.org> on 2026/07/01 20:48:42 UTC

[PR] feat(i18n): backfill Slovak (sk) translations (AI-generated, needs review) [superset]

rusackas opened a new pull request, #41640:
URL: https://github.com/apache/superset/pull/41640

   ### SUMMARY
   
   Backfills the **105 remaining untranslated entries** in the Slovak (`sk`) catalog using `scripts/translations/backfill_po.py`, which drafts translations with Claude using every other language's existing translation of the same string as cross-language disambiguation context (per `docs/developer_docs/contributing/howtos.md`).
   
   All generated strings are marked `#, fuzzy` with a `Machine-translated via backfill_po.py` attribution comment, so they are **excluded from compiled `.mo` output until a human reviewer confirms them**. Newly-filled entries preserve their format placeholders.
   
   **Note:** three entries whose English source contains literal double-quotes (e.g. `"Exact match"`, `"Cell bars"`, `"Sort by metric"`) were filled via a plain-text fallback prompt — the script's batch JSON contract mis-handles unescaped quotes in the model's response. Those three carry a `[plain-text fallback for quoted string]` attribution note. (A robustness fix for this is queued on the i18n tooling branch.)
   
   Tracked by `check_translation_regression.py` as `untranslated → fuzzy`, which is explicitly **not** a regression. Part of a per-language sweep; follows #41608 (de), #41609 (es), #41612 (lv), #41613 (fi).
   
   ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
   
   N/A — translation catalog only, no UI change until entries are reviewed and de-fuzzed.
   
   ### TESTING INSTRUCTIONS
   
   - `pybabel compile -d superset/translations -l sk` succeeds (fuzzy entries are skipped, as intended).
   - Slovak native speakers: review the `#, fuzzy` entries; correct any `msgstr` and remove the `#, fuzzy` flag + attribution comment to promote them to confirmed translations.
   
   ### ADDITIONAL INFORMATION
   
   - [ ] Has associated issue:
   - [x] Required feature flags:
   - [x] Changes UI
   - [ ] Includes DB Migration (follow approval process in [SIP-59](https://github.com/apache/superset/issues/13351))
   - [ ] Introduces new feature or API
   - [ ] Removes existing feature or API


-- 
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: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

Re: [PR] feat(i18n): backfill Slovak (sk) translations (AI-generated, needs review) [superset]

Posted by "sadpandajoe (via GitHub)" <gi...@apache.org>.
sadpandajoe merged PR #41640:
URL: https://github.com/apache/superset/pull/41640


-- 
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: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

Re: [PR] feat(i18n): backfill Slovak (sk) translations (AI-generated, needs review) [superset]

Posted by "bito-code-review[bot] (via GitHub)" <gi...@apache.org>.
bito-code-review[bot] commented on PR #41640:
URL: https://github.com/apache/superset/pull/41640#issuecomment-4861601107

   **Bito Automatic Review Skipped – PR Already Merged**<br/><br/>Bito scheduled an automatic review for this pull request, but the review was skipped because this PR was merged before the review could be run.<br/>No action is needed if you didn't intend to review it. To get a review, you can type `/review` in a comment and save it


-- 
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: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

Re: [PR] feat(i18n): backfill Slovak (sk) translations (AI-generated, needs review) [superset]

Posted by "bito-code-review[bot] (via GitHub)" <gi...@apache.org>.
bito-code-review[bot] commented on code in PR #41640:
URL: https://github.com/apache/superset/pull/41640#discussion_r3509014534


##########
superset/translations/sk/LC_MESSAGES/messages.po:
##########
@@ -2308,13 +2338,19 @@ msgstr "Auto"
 msgid "Auto Zoom"
 msgstr "Automatické priblíženie"
 
-#, python-format
+# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: de, sr,
+# sr_Latn]
+#, fuzzy, python-format

Review Comment:
   <div>
   
   
   <div id="suggestion">
   <div id="issue"><b>fuzzy flag silences Slovak translation</b></div>
   <div id="fix">
   
   This entry ("Auto refresh paused (set to %s seconds)") carries a #, fuzzy flag, which causes gettext to silently fall back to the English msgid at runtime. Slovak-speaking users will see the untranslated English string instead of the confirmed msgstr "Automatické obnovenie pozastavené (nastavené na %s sekúnd)". Remove the #, fuzzy marker so the translation is active.
   </div>
   
   
   </div>
   
   
   
   
   <small><i>Code Review Run #808eeb</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/sk/LC_MESSAGES/messages.po:
##########
@@ -2308,13 +2338,19 @@ msgstr "Auto"
 msgid "Auto Zoom"
 msgstr "Automatické priblíženie"
 
-#, python-format
+# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: de, sr,
+# sr_Latn]
+#, fuzzy, python-format
 msgid "Auto refresh paused (set to %s seconds)"
-msgstr ""
+msgstr "Automatické obnovenie pozastavené (nastavené na %s sekúnd)"
 
-#, python-format
+# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: de, sr,
+# sr_Latn]
+#, fuzzy, python-format

Review Comment:
   <div>
   
   
   <div id="suggestion">
   <div id="issue"><b>fuzzy flag silences Slovak translation</b></div>
   <div id="fix">
   
   This entry ("Auto refresh paused - tab inactive (set to %s seconds)") carries a #, fuzzy flag, causing gettext to fall back to English for Slovak users. The msgstr "Automatické obnovenie pozastavené – karta je neaktívna (nastavené na %s sekúnd)" is a valid translation but remains inactive. Remove the #, fuzzy marker.
   </div>
   
   
   </div>
   
   
   
   
   <small><i>Code Review Run #808eeb</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/sk/LC_MESSAGES/messages.po:
##########
@@ -8573,8 +8702,11 @@ msgstr "Mapbox"
 msgid "Mapbox (API key required)"
 msgstr "E-mail je povinný"

Review Comment:
   <div>
   
   
   <div id="suggestion">
   <div id="issue"><b>Incorrect translation semantically</b></div>
   <div id="fix">
   
   Translation 'E-mail je povinný' ('Email is required') is completely unrelated to 'Mapbox (API key required)'. This appears to be a copy-paste error from a different entry's translation.
   </div>
   
   
   </div>
   
   
   
   
   <small><i>Code Review Run #808eeb</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/sk/LC_MESSAGES/messages.po:
##########
@@ -15667,8 +15971,13 @@ msgstr "Nemožno dekódovať hodnotu"
 msgid "Unable to encode value"
 msgstr "Nemožno kódovať hodnotu"
 
+# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: de, sr,
+# sr_Latn]
+#, fuzzy
 msgid "Unable to fetch semantic views. Check the layer configuration."
 msgstr ""
+"Nepodarilo sa načítať sémantické zobrazenia. Skontrolujte konfiguráciu "
+"vrstvy."

Review Comment:
   <div>
   
   
   <div id="suggestion">
   <div id="issue"><b>Unreviewed fuzzy translations</b></div>
   <div id="fix">
   
   All 7 entries carry `#, fuzzy` flags and `# Machine-translated via backfill_po.py` attribution comments — they are unreviewed machine translations. Per the documented workflow in `howtos.md` (lines 403–410), these attribution comments and `#, fuzzy` flags must be removed after human verification before the `.po` file is committed. The file currently has 879 total fuzzy entries; 105 carry machine-translation attribution. These entries are sourced from non-Slovak reference languages (de, sr, sr_Latn) and should be reviewed by a Slovak speaker before the flags are cleared.
   </div>
   
   
   </div>
   
   
   
   
   <small><i>Code Review Run #808eeb</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/sk/LC_MESSAGES/messages.po:
##########
@@ -8236,8 +8349,11 @@ msgstr "Menší alebo rovno (<=)"
 msgid "Less than (<)"
 msgstr "Menší než (<)"
 
+# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: de, sr,
+# sr_Latn]
+#, fuzzy
 msgid "Liberty (OpenFreeMap)"
-msgstr ""
+msgstr "Liberty (OpenFreeMap)"

Review Comment:
   <div>
   
   
   <div id="suggestion">
   <div id="issue"><b>Source equals translation</b></div>
   <div id="fix">
   
   Source and translation are identical for 'Liberty (OpenFreeMap)'. While brand names may remain in English, this should be intentional rather than defaulting to copy-paste.
   </div>
   
   
   </div>
   
   
   
   
   <small><i>Code Review Run #808eeb</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/sk/LC_MESSAGES/messages.po:
##########
@@ -8558,13 +8679,21 @@ msgstr "Živé vykreslenie"
 msgid "Map Style"
 msgstr "Styl mapy"
 
+# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: de, sr,
+# sr_Latn]
+#, fuzzy
 msgid "MapLibre (open-source)"
-msgstr ""
+msgstr "MapLibre (open-source)"

Review Comment:
   <div>
   
   
   <div id="suggestion">
   <div id="issue"><b>Source equals translation</b></div>
   <div id="fix">
   
   Source and translation are identical for 'MapLibre (open-source)'. This may be intentional for product names but should be verified as intentional localization decision.
   </div>
   
   
   </div>
   
   
   
   
   <small><i>Code Review Run #808eeb</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/sk/LC_MESSAGES/messages.po:
##########
@@ -2838,8 +2890,11 @@ msgstr "Zrušiť"
 msgid "Cancel query on window unload event"
 msgstr "Zrušiť dotaz pri události uvolnie je okna"
 
+# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: de, es, sr,
+# sr_Latn]
+#, fuzzy

Review Comment:
   <div>
   
   
   <div id="suggestion">
   <div id="issue"><b>fuzzy flag silences Slovak translation</b></div>
   <div id="fix">
   
   This entry ("Cancellation not available due to missing abort handler") carries a #, fuzzy flag, hiding the Slovak msgstr from gettext runtime. Slovak users see the raw English string. Remove the #, fuzzy marker so the translation is active.
   </div>
   
   
   </div>
   
   
   
   
   <small><i>Code Review Run #808eeb</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/sk/LC_MESSAGES/messages.po:
##########
@@ -11092,8 +11256,11 @@ msgstr "Odstrániť"
 msgid "Revoke API Key"
 msgstr "Kľúč skupiny"

Review Comment:
   <div>
   
   
   <div id="suggestion">
   <div id="issue"><b>Quality: pre-existing wrong translation breaks cluster consistency</b></div>
   <div id="fix">
   
   The pre-existing entry for "Revoke API Key" (msgstr: "Kľúč skupiny") is clearly mistranslated — it says 'group key', not 'revoke API key'. This creates a visible inconsistency with the adjacent newly added "Revoke this API key" entry whose translation ("Odvolať tento kľúč API") is correct. Both entries belong to the same 'Revoke API' functional cluster and should be translated consistently.
   </div>
   
   
   </div>
   
   
   
   
   <small><i>Code Review Run #808eeb</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: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

Re: [PR] feat(i18n): backfill Slovak (sk) translations (AI-generated, needs review) [superset]

Posted by "bito-code-review[bot] (via GitHub)" <gi...@apache.org>.
bito-code-review[bot] commented on PR #41640:
URL: https://github.com/apache/superset/pull/41640#issuecomment-4859922579

   AI Code Review is in progress (usually takes 3 to 15 minutes unless it's a very large PR).


-- 
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: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

Re: [PR] feat(i18n): backfill Slovak (sk) translations (AI-generated, needs review) [superset]

Posted by "netlify[bot] (via GitHub)" <gi...@apache.org>.
netlify[bot] commented on PR #41640:
URL: https://github.com/apache/superset/pull/41640#issuecomment-4859941206

   ### <span aria-hidden="true">✅</span> Deploy Preview for *superset-docs-preview* ready!
   
   
   |  Name | Link |
   |:-:|------------------------|
   |<span aria-hidden="true">🔨</span> Latest commit | 7d9bf5d5605029c0b685fa999c23f4c6d5a544da |
   |<span aria-hidden="true">🔍</span> Latest deploy log | https://app.netlify.com/projects/superset-docs-preview/deploys/6a457d2a91b360000834161a |
   |<span aria-hidden="true">😎</span> Deploy Preview | [https://deploy-preview-41640--superset-docs-preview.netlify.app](https://deploy-preview-41640--superset-docs-preview.netlify.app) |
   |<span aria-hidden="true">📱</span> Preview on mobile | <details><summary> Toggle QR Code... </summary><br /><br />![QR Code](https://app.netlify.com/qr-code/eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJ1cmwiOiJodHRwczovL2RlcGxveS1wcmV2aWV3LTQxNjQwLS1zdXBlcnNldC1kb2NzLXByZXZpZXcubmV0bGlmeS5hcHAifQ.5NGHstqI6ojPUdVdzuJBjVskUuwO3OpTrjYsEahV2vQ)<br /><br />_Use your smartphone camera to open QR code link._</details> |
   |<span aria-hidden="true">🤖</span> Make changes | [Run an agent on this branch](https://app.netlify.com/projects/superset-docs-preview/agent-runs#DYrBDYAgEMB2ubdEEUR0A8c4BCMRgQAfY9zd66tp-kIu6c4NVoAOTMG4n-SHw9Z7riOrF2tUa8DmU6w0WZdDejZLm0I5zXbEhRuhBkILyRVH-H4) |
   ---
   <!-- [superset-docs-preview Preview](https://deploy-preview-41640--superset-docs-preview.netlify.app) -->
   _To edit notification comments on pull requests, go to your [Netlify project configuration](https://app.netlify.com/projects/superset-docs-preview/configuration/notifications#deploy-notifications)._


-- 
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: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

Re: [PR] feat(i18n): backfill Slovak (sk) translations (AI-generated, needs review) [superset]

Posted by "codecov[bot] (via GitHub)" <gi...@apache.org>.
codecov[bot] commented on PR #41640:
URL: https://github.com/apache/superset/pull/41640#issuecomment-4860809618

   ## [Codecov](https://app.codecov.io/gh/apache/superset/pull/41640?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
   :white_check_mark: All modified and coverable lines are covered by tests.
   :white_check_mark: Project coverage is 55.80%. Comparing base ([`e58ce1c`](https://app.codecov.io/gh/apache/superset/commit/e58ce1cf39d1a0aa5d597ace53d1c0792bae56df?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`de1f43c`](https://app.codecov.io/gh/apache/superset/commit/de1f43cc27c6bb3d0c8adc1e21b40b2b86eeb8ae?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
   
   <details><summary>Additional details and impacted files</summary>
   
   
   
   ```diff
   @@            Coverage Diff             @@
   ##           master   #41640      +/-   ##
   ==========================================
   + Coverage   55.07%   55.80%   +0.72%     
   ==========================================
     Files        2673     2673              
     Lines      147634   147609      -25     
     Branches    34090    34078      -12     
   ==========================================
   + Hits        81312    82367    +1055     
   + Misses      65916    64496    -1420     
   - Partials      406      746     +340     
   ```
   
   | [Flag](https://app.codecov.io/gh/apache/superset/pull/41640/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
   |---|---|---|
   | [hive](https://app.codecov.io/gh/apache/superset/pull/41640/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `39.06% <ø> (ø)` | |
   | [presto](https://app.codecov.io/gh/apache/superset/pull/41640/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `40.61% <ø> (?)` | |
   | [python](https://app.codecov.io/gh/apache/superset/pull/41640/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `40.67% <ø> (+1.60%)` | :arrow_up: |
   
   Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more.
   </details>
   
   [:umbrella: View full report in Codecov by Harness](https://app.codecov.io/gh/apache/superset/pull/41640?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).   
   :loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
   <details><summary> :rocket: New features to boost your workflow: </summary>
   
   - :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems.
   - :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
   </details>


-- 
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: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org