🌐 Update German translations for recently updated docs (using LLMs)#13930
Closed
🌐 Update German translations for recently updated docs (using LLMs)#13930
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
nilslindemann
pushed a commit
to nilslindemann/fastapi
that referenced
this pull request
Aug 4, 2025
…#13929 (translations-de), fastapi#13930 (update-translations-de-ai)
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.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
ec64e6e to
7aa880a
Compare
This comment was marked as outdated.
This comment was marked as outdated.
7aa880a to
15a52d4
Compare
This comment was marked as outdated.
This comment was marked as outdated.
Contributor
|
@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).
YuriiMotov
reviewed
Aug 6, 2025
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Contributor
nilslindemann
approved these changes
Aug 17, 2025
Contributor
nilslindemann
left a comment
There was a problem hiding this comment.
This is good 👍 and can be merged as is, if you want, I will add the permalinks in a later PR.
Contributor
|
This pull request has a merge conflict that needs to be resolved. |
Contributor
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🌐 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:
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.