-
Notifications
You must be signed in to change notification settings - Fork 17
Add translation example to LaTeX to Word conversion notebook #878
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
base: main
Are you sure you want to change the base?
Add translation example to LaTeX to Word conversion notebook #878
Conversation
|
Thank you so much for contributing to Blueprints! Now that you've created your pull request, please don't go away; take a look at the bottom of this page for the automated checks that should already be running. If they pass, great! If not, please click on 'Details' and see if you can fix the problem they've identified. A maintainer should be along shortly to review your pull request and help get it added! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds a new example (1b) to the LaTeX to Word conversion notebook demonstrating how to translate LaTeX text before converting it to Word format using the TranslateLatex module. This enhancement enables users to localize formula text for document generation in different languages.
- Added Example 1b showing LaTeX translation with Dutch language conversion
- Cleared execution outputs from existing Examples 2, 3, and 4
- Updated cell IDs and execution counts to accommodate the new example
Comments suppressed due to low confidence (1)
docs/examples/latex/latex_to_word.ipynb:1
- The new translation functionality introduced in Example 1b lacks test coverage. Consider adding tests to verify that the TranslateLatex integration works correctly with the LatexToWordConverter, particularly testing that translated LaTeX maintains proper formatting and can be successfully converted to Word documents.
{
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #878 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 402 402
Lines 12602 12602
=========================================
Hits 12602 12602 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
egarciamendez
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At this moment we are in the process of moving away from jupyter notebooks for writing examples. Could you please use markdown instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
sure |
Description
Add translation example to LaTeX to Word conversion notebook
Fixes #877
Type of change
Checklist: