Skip to content

Conversation

@NoB0
Copy link
Contributor

@NoB0 NoB0 commented Nov 22, 2023

What does this PR do?

Translated the autoclass_tutorial.md file of the documentation to French.

Part of #21456

Thank you in advance for your review.

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you read the contributor guideline,
    Pull Request section?
  • Was this discussed/approved via a Github issue or the forum? Please add a link
    to it if that's the case.
  • Did you make sure to update the documentation with your changes? Here are the
    documentation guidelines, and
    here are tips on formatting docstrings.
  • Did you write any new necessary tests?

Who can review?

French speaking contributors.

@NoB0 NoB0 mentioned this pull request Nov 22, 2023
10 tasks
@NoB0 NoB0 changed the title Documentation/fr translation autoclass tutorial [i18n-fr] Translate autoclass tutorial to French Nov 23, 2023
Copy link
Member

@stevhliu stevhliu left a comment

Choose a reason for hiding this comment

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

Thanks for the translation!

I think your toctree may be out of order though. The Task Guides come after the Tutorials instead of the Developer Guides.

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Copy link
Member

@stevhliu stevhliu left a comment

Choose a reason for hiding this comment

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

Very nice, LGTM! Feel free to mark the PR for review whenever you're ready. Pinging @ArthurZucker for a quick review of the content :)

Copy link
Collaborator

@ArthurZucker ArthurZucker left a comment

Choose a reason for hiding this comment

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

Looks good but let's not use point de control it's a word for word translation but does not sound correct to me (unless that is what's used for all the other translations)


# Chargement d'instances pré-entraînées avec une AutoClass

Avec autant d'architectures Transformer différentes, il peut être difficile d'en créer une pour votre point de contrôle. Dans l'idée de créer une librairie facile, simple et flexible à utiliser, 🤗 Transformers fournit une `AutoClass` qui infère et charge automatiquement l'architecture correcte à partir d'un point de contrôle donné. La fonction `from_pretrained()` vous permet de charger rapidement un modèle pré-entraîné pour n'importe quelle architecture afin que vous n'ayez pas à consacrer du temps et des ressources à l'entraînement d'un modèle à partir de zéro. Produire un tel code indépendant du point de contrôle signifie que si votre code fonctionne pour un point de contrôle, il fonctionnera avec un autre point de contrôle - tant qu'il a été entraîné pour une tâche similaire - même si l'architecture est différente.
Copy link
Collaborator

Choose a reason for hiding this comment

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

point de control is necessarly wrong 😅 let's check what other translation use instead of this but it sounds really off.

Copy link
Contributor Author

@NoB0 NoB0 Dec 6, 2023

Choose a reason for hiding this comment

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

Agreed, I modified it to ensemble de poids (per your comment on my previous question). I also added in parentheses the standard terms in English as it may be possible that they are used as-is in French. Please check.

@NoB0 NoB0 marked this pull request as ready for review December 6, 2023 16:44
@ArthurZucker ArthurZucker merged commit 06f5616 into huggingface:main Dec 7, 2023
@ArthurZucker
Copy link
Collaborator

Thanks for the translation!

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.

4 participants