Skip to content

Fix: Refactor parser config handling and add GraphRAG defaults #8778

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

Merged

Conversation

asiroliu
Copy link
Contributor

@asiroliu asiroliu commented Jul 10, 2025

What problem does this PR solve?

Type of change

  • Bug Fix (non-breaking change which fixes an issue)

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. 🌈 python Pull requests that update Python code 🐞 bug Something isn't working, pull request that fix bug. labels Jul 10, 2025
@asiroliu asiroliu added ci Continue Integration wip labels Jul 10, 2025
@asiroliu asiroliu changed the title Fix: Implement deep merge for ParserConfig to handle default and custom settings Fix: Refactor parser config handling and add GraphRAG defaults Jul 22, 2025
@asiroliu asiroliu force-pushed the fix/ParserConfig_handle_default_settings branch from 13c8f0b to 09cf010 Compare July 22, 2025 08:07
@asiroliu asiroliu removed the wip label Jul 22, 2025
- Update `get_parser_config` to merge provided configs with defaults
- Add GraphRAG configuration defaults for all chunk methods
- Make raptor and graphrag fields non-nullable in ParserConfig schema
- Update related test cases to reflect config changes
- Ensure backward compatibility while adding new GraphRAG support
- infiniflow#8396
@asiroliu asiroliu force-pushed the fix/ParserConfig_handle_default_settings branch from 09cf010 to 00f08a9 Compare July 22, 2025 08:11
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:S This PR changes 10-29 lines, ignoring generated files. labels Jul 22, 2025
@KevinHuSh KevinHuSh merged commit 0020c50 into infiniflow:main Jul 23, 2025
1 of 2 checks passed
@asiroliu asiroliu deleted the fix/ParserConfig_handle_default_settings branch July 23, 2025 05:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug Something isn't working, pull request that fix bug. ci Continue Integration 🌈 python Pull requests that update Python code size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants