-
Notifications
You must be signed in to change notification settings - Fork 3.5k
feat: Kuzu Graph DB integration and adds bi-lingual (english, chinese) support to graph testing suite #1763
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?
Conversation
|
Thank you for your congratulations and efforts. As LightRAG evolves, each storage implementation necessitates corresponding upgrades. Currently, we lack the resources to consistently track and maintain the advancements and performance optimizations for every storage type. To avoid impeding the overall progress of the LightRAG system, LightRAG will temporarily discontinue the integration of new storage implementations. |
|
@loganpowell - is this currently working? I want to integrate KuzuDB into my project too! |
|
@voycey, it passes all the graph operation tests, so it should work. However I haven't tested it e2e. |
|
Ive merged it into my project to test - Ive also asked the maintainers of LightRAG to merge this for their next release so fingers crossed! |
|
You may keep this PR open and frequently rebase it against the |
|
I am actively using this, I'll report back as I go deeper with it but so far it looks good! Would appreciate this being merged into the main branch @danielaskdd, the networkx option isn't the best approach! |
|
v1.4.9rc1 has been released. All graph databases must implement the following two new functions:
@loganpowell could you please update the KuzuDB implementation to align with the new requirements? |
…port) - Add pytest asyncio configuration in pyproject.toml with auto mode - Create comprehensive tests/conftest.py with dynamic storage fixture - Implement robust test environment setup and cleanup for KuzuDB - Add mock embedding function for consistent testing - Fix all async test compatibility issues (18 tests now passing) - Resolve missing storage fixture errors across all test files - Add conditional server availability checks for Ollama tests - Implement @requires_server decorator to gracefully skip tests when external dependencies unavailable - Fix import paths in conftest.py (lightrag.kg.* module structure) - Add comprehensive KuzuDB integration tests - Enhance kuzu_impl.py with better error handling and edge case management - Update __init__.py exports for proper module access - Add new translation implementation tests and utilities - Create demo translation examples for testing multilingual support - Add comprehensive documentation for KuzuDB implementation - Include translation status tracking and implementation summaries - All storage tests (graph + KuzuDB) now pass consistently - External server dependencies gracefully handled via skip markers - Proper resource cleanup after test execution - Better error messages and debugging support - ✅ 18 tests passing (graph storage, KuzuDB, translations) - ⏭️ 10 tests skipped (Ollama server not available) - 🔧 Async test execution fully supported - 📚 Comprehensive test documentation added
formatting
|
KuzuDB Integration
Test Infrastructure Overhaul
Core Test Framework
Test Suite Enhancements
Test Coverage Expansion
Robustness Improvements
Results