Skip to content

🌐 Update German translations for recently updated docs (using LLMs)#13930

Closed
tiangolo wants to merge 6 commits intomasterfrom
update-translations-de-ai
Closed

🌐 Update German translations for recently updated docs (using LLMs)#13930
tiangolo wants to merge 6 commits intomasterfrom
update-translations-de-ai

Conversation

@tiangolo
Copy link
Member

@tiangolo tiangolo commented Jul 26, 2025

🌐 Update German translations for recently updated docs (using LLMs)

This is done with the German LLM prompt file (#13931), if there are things that are wrong in this PR, things that shouldn't be updated or things missing, it would be better to tweak that prompt file to instruct the LLM to better handle those cases than to review this PR directly.


These are the existing translations that have been updated in the original files in English since the last update in German.

These are probably the most delicate, because the prompt is adjusted to try both:

  • Update the existing translation to cover whatever is new in the original docs in English.
  • Keep what's already fine, avoid just rewriting everything every time, otherwise this could run over and over again generating something completely new each time, and any update would make the diff giant, instead of focusing on what actually changed.

Both objectives are at odds, one would expect the LLM to change more things, the other would expect the LLM to change fewer things.

There are some pages that have changed quite a bit in the original in English since they were translated, so I imagine there's a chance the LLM would end up defaulting to changing more in those files.

There are also some files that have changed only in specific sections, for example adding or removing things at the end. It seems the LLM can get these fairly well, only adding (or removing) that final section but respecting the rest.

If there are some files that are changed too much unnecessarily, or not working well, maybe a note with which files are okay to merge would work, so that I can remove the bad ones to tweak them further and keep the good translation files.

@github-actions github-actions bot added the lang-all Translations label Jul 26, 2025
@github-actions

This comment was marked as outdated.

@tiangolo tiangolo marked this pull request as ready for review July 26, 2025 21:03
@tiangolo tiangolo added awaiting-review lang-de German translations labels Jul 26, 2025
nilslindemann pushed a commit to nilslindemann/fastapi that referenced this pull request Aug 4, 2025
nilslindemann added a commit to nilslindemann/fastapi that referenced this pull request Aug 4, 2025
This time I regenerated the translations, using the existing translations from fastapi#13930, and some wordlists, not from scratch, as the previous commit. There was one case where the AI did not remove a paragraph, which was removed from the english text, see query-params-str-validations.md, but otherwise this went smooth. Rarely were manual changes necessary. Is now in sync with the English docs.

You may apply this commit to fastapi#13930, Regeneration is not necessary.
This time I regenerated the translations, using the existing translations from #13930, and some wordlists, not from scratch, as the previous commit. There was one case where the AI did not remove a paragraph, which was removed from the english text, see query-params-str-validations.md, but otherwise this went smooth. Rarely were manual changes necessary. Is now in sync with the English docs.

You may apply this commit to #13930, Regeneration is not necessary.
@github-actions

This comment was marked as outdated.

Copy link
Contributor

@nilslindemann nilslindemann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good 👍

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@YuriiMotov YuriiMotov force-pushed the update-translations-de-ai branch from ec64e6e to 7aa880a Compare August 5, 2025 08:13
@github-actions

This comment was marked as outdated.

@YuriiMotov YuriiMotov force-pushed the update-translations-de-ai branch from 7aa880a to 15a52d4 Compare August 5, 2025 08:56
@github-actions

This comment was marked as outdated.

@nilslindemann
Copy link
Contributor

nilslindemann commented Aug 5, 2025

@YuriiMotov, I can add the anchors in the rest of the documents too, if you want. Edit: already did, see here Edit 2: this comment can be marked as outdated.

nilslindemann added a commit to nilslindemann/fastapi that referenced this pull request Aug 5, 2025
Having the changes from the previous three commits applied.

And a few other fixes found by the AI.

I have checked it manually.

Please apply to fastapi#13930 (update-translations-de-ai).
@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions
Copy link
Contributor

github-actions bot commented Aug 8, 2025

📝 Docs preview for commit b60b0e0 at: https://a005cd42.fastapitiangolo.pages.dev

Modified Pages

Copy link
Contributor

@nilslindemann nilslindemann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is good 👍 and can be merged as is, if you want, I will add the permalinks in a later PR.

@github-actions github-actions bot added the conflicts Automatically generated when a PR has a merge conflict label Sep 5, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Sep 5, 2025

This pull request has a merge conflict that needs to be resolved.

@nilslindemann
Copy link
Contributor

#13931, #13929, and #13930 (this one) can be closed, as I have applied them in #14015 and their changes will be committed when that one is committed. Also, I have resolved merge conflicts there already.

@YuriiMotov YuriiMotov closed this Sep 5, 2025
@YuriiMotov YuriiMotov deleted the update-translations-de-ai branch September 24, 2025 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved-1 conflicts Automatically generated when a PR has a merge conflict lang-all Translations lang-de German translations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants