-
Notifications
You must be signed in to change notification settings - Fork 937
feat: implement auto-translation task for multilingual content #1353
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
Closed
Closed
Changes from 1 commit
Commits
Show all changes
66 commits
Select commit
Hold shift + click to select a range
c1e8ba2
feat: implement auto-translation task for multilingual content
subhash-0000 1ab9320
fix: address code review feedback
subhash-0000 8feab1d
docs: update cognify pipeline documentation
subhash-0000 ea035d9
Address CodeRabbit review feedback
subhash-0000 cb7f9c3
Address final CodeRabbit review comments
subhash-0000 d945381
Address maintainer feedback - implement pluggable translation system
subhash-0000 376aaa1
Address CodeRabbit review feedback
subhash-0000 9f2079c
Enhance docstrings and fix remaining search call
subhash-0000 8d83c54
Address remaining CodeRabbit feedback
subhash-0000 59c53e2
Final CodeRabbit fixes - no more iterations needed
subhash-0000 5c2fd67
Complete remaining CodeRabbit fixes
subhash-0000 ab002a4
Address all CodeRabbit feedback - final cleanup
subhash-0000 11c6dcf
Address final CodeRabbit nitpicks for robustness
subhash-0000 5097abd
FINAL: Complete comprehensive audit addressing ALL issues
subhash-0000 4e21f03
Apply final robustness improvements per CodeRabbit review
subhash-0000 f309817
feat: address final CodeRabbit feedback for production readiness
subhash-0000 08f62f5
refactor: address priority CodeRabbit nitpicks for code quality
subhash-0000 c7ccc1d
fix: Address all CodeRabbit feedback and add comprehensive tests
subhash-0000 286aff7
fix: Address final 2 CodeRabbit actionable comments
subhash-0000 cca3396
fix: Address key CodeRabbit nitpicks for approval
subhash-0000 115d99b
Address CodeRabbit feedback: Fix imports, validation, and clean up code
subhash-0000 18f70b1
fix: Address remaining CodeRabbit feedback
subhash-0000 98af2ed
fix: resolve all CodeRabbit feedback for auto-translation feature
subhash-0000 bff993d
fix: Address code review comments for translation system
subhash-0000 b11fef9
fix: resolve critical language normalization and test validation issues
subhash-0000 94a013c
Improve translation functionality and code structure
subhash-0000 0bac83b
chore: consolidate cognify and polish translation\n- Remove duplicate…
subhash-0000 d304b0c
chore(api): remove duplicate api/v1/cognify/cognify.py; standardize o…
subhash-0000 4f6a541
translation: normalize detection confidence, guard empty translations…
subhash-0000 5fa262e
translation: allow fallback by returning None from NoOp.detect_langua…
subhash-0000 88efbd5
translation: normalize language codes and use math.isnan; OpenAI: set…
subhash-0000 9187fd8
translation: refactor translate_content into helpers to reduce statem…
subhash-0000 f6f9dc4
translation: canonical provider name in metadata; preflight init erro…
subhash-0000 8d446bc
translation: remove stray duplicate return in translate_content; fix …
subhash-0000 003ec79
fix: remove duplicate tasks/translation module to resolve import shad…
subhash-0000 82985c9
new updates
subhash-0000 9f6b2dc
📝 Add docstrings to `auto-translate-task`
coderabbitai[bot] 94c53c1
refactor: Refactor translation task for maintainability and determinism
subhash-0000 c01b63e
feat: Add docstrings from bot PR
subhash-0000 9867b35
feat: Merge docstrings and resolve conflicts
subhash-0000 888f745
fix: Address review comments and resolve merge conflicts
subhash-0000 4741a9f
feat: implement all review suggestions
subhash-0000 2ff6f94
fix: address nitpick comments from review
subhash-0000 8af2f4a
fix: address final round of review comments
subhash-0000 5c71402
fix: address all outstanding code review comments
subhash-0000 0817bd1
fix: comprehensive code review improvements and provider fallback
subhash-0000 05f7986
final fixes
subhash-0000 5aa9a74
Fix TRY300 linting warnings - move statements to else blocks
subhash-0000 6efb9df
Comprehensive translation system improvements
subhash-0000 2b72e1e
refactor: apply cleaner helper function approach to resolve R0914 com…
subhash-0000 f8b6e0e
fix: address security and linting issues in translation module
subhash-0000 52aca87
fix: resolve technical issues and optimize provider performance
subhash-0000 c89eda0
fix: address function complexity and prompt optimization
subhash-0000 ce343c2
Fix recurring patterns: async cancellation, timeouts, and defensive p…
subhash-0000 67fb9aa
Apply systematic pattern fixes: prevent metadata conflicts and race c…
subhash-0000 c256b67
Refactor translation models, fix DataPoint import, and retarget PR to…
subhash-0000 d80f51e
Merge branch 'dev' into auto-translate-task
subhash-0000 615000a
Fix import and undefined references in cognify.py for clean execution
subhash-0000 085d505
observability: restore get_observe to return langfuse observe or no-o…
subhash-0000 2f14ec2
observability: remove redundant observe None fallback; add translatio…
subhash-0000 d480614
translation: clean imports, add provider error classes, and tidy tran…
subhash-0000 858173b
translation: finalize detection_provider wiring and exports
subhash-0000 f70a2a1
chore: pin httpx/httpcore for googletrans compatibility; add smoke Ge…
subhash-0000 cb4bc17
chore: unregister untested translation providers (google/azure/llm) p…
subhash-0000 aa1c17c
chore: remove untested translation providers (google/azure/llm) and s…
subhash-0000 03488f3
fix: remove duplicate imports and resolve import errors in translatio…
subhash-0000 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
docs: update cognify pipeline documentation
- Add translation step to processing pipeline documentation - Document translation_provider parameter with accepted values - Explain OpenAI API key requirement for openai provider - Clarify cross-language search capability Addresses remaining documentation feedback from code review.
- Loading branch information
commit 8feab1d65b6fa6a41730df275939984b7474fc3d
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
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
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.
Uh oh!
There was an error while loading. Please reload this page.