Skip to content

add action to translate readme #151

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

add action to translate readme #151

wants to merge 12 commits into from

Conversation

pelikhan
Copy link
Member

@pelikhan pelikhan commented Jul 9, 2025

This pull request configure AI-generated translations for french (only lessons and readme) using github actions and github models. Large files are still a bit of a problem, good start.

  • the github action triggers on main when markdown files are modified.
  • it performs as much translations as possible and pushes the result into main

@pelikhan pelikhan marked this pull request as ready for review July 12, 2025 13:09
Copy link
Member Author

@pelikhan pelikhan left a comment

Choose a reason for hiding this comment

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

First iteration added. Some of the prompts are reaching 16k tokens and hitting the upper limit. Chunking documents will certainly help with this.

@pelikhan pelikhan requested a review from sinedied July 12, 2025 13:16
Copy link
Contributor

@sinedied sinedied left a comment

Choose a reason for hiding this comment

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

Reviewed french translations and it's quite good!

Few questions though:

  • Is there a global config file that we can use to set global translations rules, like for example avoid translating specific words?
  • Is the cache shared between file, so an exact same paragraph in different files would be translated the same, for consistency? (and less token usage 🙂 )

Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a way to exclude specific words to be translated? Like for example "prompt"? Do you think we can use frontmatter metadata for that?

Consultez l'[application d'exemple](/app/README.md) pour commencer.
> \[!ASTUCE]
Copy link
Contributor

Choose a reason for hiding this comment

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

This [!TIP] should not be translated. A good candidate for global rules

Copy link
Member Author

Choose a reason for hiding this comment

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

Right this is a bug. We should be handling the GFM annotation specifically and ignore them.

Copy link
Member Author

Choose a reason for hiding this comment

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

There's a few broken translations in the cache. I will remove them and rerun the translation.

Copy link
Contributor

Choose a reason for hiding this comment

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

Some untranslated text parts here, might need to see if it's the model and how it can be improved

Copy link
Member Author

Choose a reason for hiding this comment

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

sometimes, the validation refuses the translation.

Copy link
Contributor

Choose a reason for hiding this comment

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

Can these refusals be tagged in the translation cache?


## Récit - De Charybde en Scylla

> > [!NOTE]
Copy link
Contributor

Choose a reason for hiding this comment

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

Double > here, not sure why? maybe trying to recompile the MD node after translation to see the same node structure was kept might help with issues like these?


## Ressources d'auto-apprentissage

* [\* \[Série de vidéos sur l'intelligence artificielle générative avec JavaScript\](https://aka.ms/genai-js)](https://aka.ms/genai-js)
Copy link
Contributor

Choose a reason for hiding this comment

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

Seems something wrong there with the link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants