Stirling-PDF icon indicating copy to clipboard operation
Stirling-PDF copied to clipboard

Update and improve Hungarian translations using Claude 3.5 Sonnet

Open PeterDaveHello opened this issue 1 year ago • 6 comments

Description

Leverage Claude 3.5 Sonnet from Anthropic, a frontier LLM model, to enhance translation quality. The previous use of Qwen2.5 7b, a smaller and less advanced AI model, may have affected the quality and user experience(#2164). Typically, relying solely on LLMs for translation without verification is not preferred. However, given these concerns, using a frontier LLM model along with references from other language versions should lead to improved results, even for a language I don't even understand.

I also use the powerful frontier OpenAI O1 and Google Gemini-1.5-Pro for proofreading, and manually confirmed line breaks to facilitate easier contributions and improvements by others.

Checklist

  • [x] I have read the Contribution Guidelines
  • [ ] I have performed a self-review of my own code
  • [ ] I have attached images of the change if it is UI based
  • [ ] I have commented my code, particularly in hard-to-understand areas
  • [ ] If my code has heavily changed functionality I have updated relevant docs on Stirling-PDFs doc repo
  • [ ] My changes generate no new warnings
  • [ ] I have read the section Add New Translation Tags (for new translation tags only)

PeterDaveHello avatar Dec 22 '24 16:12 PeterDaveHello

@Frooodle, if you need help with the review, maybe https://coderabbit.ai/ could be helpful.

Reference case: https://github.com/transmission-remote-gui/transgui/pull/1479#issuecomment-2389692543

I'm not affiliated with coderabbit, just thought it may help, from my personal experience.

PeterDaveHello avatar Dec 22 '24 16:12 PeterDaveHello

📃 File Check: messages_hu_HU.properties

  1. Test Status:Passed
  2. Test Status:Passed

✅ Overall Check Status: Success

@Frooodle

Ludy87 avatar Dec 22 '24 16:12 Ludy87

/deploypr

Ludy87 avatar Dec 22 '24 17:12 Ludy87

🚀 PR Test Deployment

Your PR has been deployed for testing!

🔗 Test URL: http://185.252.234.121:2549

This deployment will be automatically cleaned up when the PR is closed.

github-actions[bot] avatar Dec 22 '24 17:12 github-actions[bot]

@PeterDaveHello can you merge latest changes into your branch

Frooodle avatar Dec 22 '24 18:12 Frooodle

@Frooodle I rebased on the latest main branch, please take a look.

PeterDaveHello avatar Dec 23 '24 15:12 PeterDaveHello

Sorry was going to merge today but saw some conflicts it wont let me fix it from my side due to your security settings

Frooodle avatar Jan 02 '25 15:01 Frooodle

@PeterDaveHello could you update your branch so we can merge the updates, thanks.

Ludy87 avatar Jan 08 '25 16:01 Ludy87

Updated.

PeterDaveHello avatar Jan 12 '25 10:01 PeterDaveHello

🚀 Translation Verification Summary

🔄 Reference Branch: main-branch-messages_en_GB.properties

📃 File Check: messages_hu_HU.properties

  1. Test Status:Passed
  2. Test Status:Passed

✅ Overall Check Status: Success

Thanks @PeterDaveHello for your help in keeping the translations up to date.

github-actions[bot] avatar Jan 12 '25 10:01 github-actions[bot]