EC-RAG 26.01 release with code enhancement and bug fix#2409
Merged
Conversation
Signed-off-by: Yongbozzz <yongbo.zhu@intel.com>
Dependency ReviewThe following issues were found:
VulnerabilitiesEdgeCraftRAG/edgecraftrag/requirements.txtScanned Files
|
Contributor
There was a problem hiding this comment.
Pull request overview
This PR introduces enhancements to the EC-RAG system, focusing on supporting multiple generators per pipeline, improving agent creation logic, and implementing various UI and performance improvements. The changes enable pipelines to have both ChatQnA and FreeChat generators, enhance benchmark performance tracking, and add retry/replace functionality for failed file uploads in the knowledge base.
Changes:
- Enhanced pipeline architecture to support multiple generators (ChatQnA and FreeChat) instead of a single generator
- Improved agent creation logic to automatically create FreeChat generators when needed
- Added retry/replace functionality for failed file uploads in the knowledge base UI
- Enhanced benchmark performance tracking with improved metrics collection and timing accuracy
Reviewed changes
Copilot reviewed 48 out of 72 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| EdgeCraftRAG/ui/vue/src/views/settings/index.vue | Added conditional rendering for Pipeline and Agent tabs to optimize component lifecycle |
| EdgeCraftRAG/ui/vue/src/views/settings/components/Pipeline/components/UpdateDialog/Indexer.vue | Enhanced endpoint validation logic and error handling for indexer and vLLM configurations |
| EdgeCraftRAG/ui/vue/src/views/settings/components/Pipeline/components/UpdateDialog/Generator.vue | Refactored generator configuration to support multiple generators with independent state management |
| EdgeCraftRAG/ui/vue/src/views/settings/components/Pipeline/components/UpdateDialog/CreateDialog.vue | Added validation state tracking to prevent proceeding with untested endpoints |
| EdgeCraftRAG/ui/vue/src/views/settings/components/Pipeline/components/Table.vue | Updated table display to show generator types and fixed pagination initialization |
| EdgeCraftRAG/ui/vue/src/views/settings/components/Pipeline/components/DetailDrawer.vue | Updated to handle multiple generators in pipeline details display |
| EdgeCraftRAG/ui/vue/src/views/settings/components/Pipeline/columnsList.ts | Modified table columns to display generator type instead of inference type |
| EdgeCraftRAG/ui/vue/src/views/settings/components/Agent/components/UpdateDialog.vue | Removed pipeline type filtering and debug logging |
| EdgeCraftRAG/ui/vue/src/views/settings/components/Agent/components/Table.vue | Fixed pagination initialization and added total count display |
| EdgeCraftRAG/ui/vue/src/views/settings/components/Agent/columnsList.ts | Changed agent table to display pipeline name instead of pipeline ID |
| EdgeCraftRAG/ui/vue/src/views/chatbot/components/KnowledgeBase/KnowledgeDetail.vue | Added comprehensive retry/replace/ignore functionality for failed file uploads |
| EdgeCraftRAG/ui/vue/src/utils/validate.ts | Updated service address validation to support HTTPS protocol |
| EdgeCraftRAG/ui/vue/src/utils/notification.ts | Added default notification duration of 3 seconds |
| EdgeCraftRAG/ui/vue/src/i18n/zh.ts | Added Chinese translations for retry, replace, ignore actions and URL validation tip |
| EdgeCraftRAG/ui/vue/src/i18n/en.ts | Added English translations for retry, replace, ignore actions and URL validation tip |
| EdgeCraftRAG/tools/quick_start.sh | Added virtual environment setup and updated package installation commands |
| EdgeCraftRAG/edgecraftrag/requirements.txt | Updated package versions for langchain-core and llama-index-core |
| EdgeCraftRAG/edgecraftrag/controllers/pipelinemgr.py | Added pipeline caching mechanism and generator type filtering |
| EdgeCraftRAG/edgecraftrag/controllers/agentmgr.py | Enhanced agent creation to auto-create FreeChat generators |
| EdgeCraftRAG/edgecraftrag/components/pipeline.py | Refactored to support multiple generators with improved benchmark tracking |
| EdgeCraftRAG/edgecraftrag/components/generator.py | Added generator cloning functionality and enhanced streaming with benchmark support |
| EdgeCraftRAG/edgecraftrag/components/benchmark.py | Improved benchmark data management and vLLM metrics collection |
| EdgeCraftRAG/edgecraftrag/base.py | Separated BenchType enum from CompType for better categorization |
| EdgeCraftRAG/edgecraftrag/api/v1/pipeline.py | Updated pipeline API to handle multiple generators and improved benchmark response |
| EdgeCraftRAG/edgecraftrag/api/v1/chatqna.py | Updated to use generator type when accessing pipeline generators |
| EdgeCraftRAG/edgecraftrag/api/v1/agent.py | Added pipeline status management for agent bound pipelines |
Comments suppressed due to low confidence (1)
EdgeCraftRAG/ui/vue/src/components/TableColumns.vue:1
- The CSS color syntax
rgb(from var(...) r g b / 0.4)is part of CSS Color Module Level 5 which may not be widely supported. Consider using a more compatible approach like defining the opacity value separately or using rgba() with a fallback.
<template>
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
for more information, see https://pre-commit.ci
ZePan110
approved these changes
Jan 28, 2026
lvliang-intel
approved these changes
Jan 29, 2026
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
Code enhancement and known issue fix :
Issues
n/a
Type of change
List the type of change like below. Please delete options that are not relevant.
Dependencies
n/a
Tests