Skip to content

New Productivity Suite react UI and Bug Fixes #1834

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
merged 36 commits into from
Apr 21, 2025

Conversation

sgurunat
Copy link
Contributor

Description

This Pull Request introduces a new React-based user interface for the ProductivitySuite along with issue fixes for Docsum, Data Management and Code Generation.

Type of change

List the type of change like below. Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds new functionality)
  • Breaking change (fix or feature that would break existing design and interface)
  • Others (enhancement, documentation, validation, etc.)

Changes

  • New React UI: Replaced the existing UI with a newly designed React-based user interface.

  • Updated Screenshots: Replaced outdated screenshots in the documentation with new ones reflecting the updated UI.

  • Documentation Updates: Revised the documentation to align with the new UI and its features.

  • Docker Compose File: Updated the docker-compose.yml file to ensure compatibility with the new UI.

  • README Updates: Updated the README file to reflect the changes in the UI and provide accurate setup instructions.

  • Bug Fixes: Fixed issues with Document Summarization, Data Management and Code Generation.

Tests

  • Verified the new UI functionality locally.
  • Ensured the updated Docker Compose configuration works as expected.
  • Reviewed all documentation changes for accuracy.

sgurunat and others added 6 commits April 15, 2025 10:06
…ummarize and Code Generation

Signed-off-by: Gurunath S <gurunath.s@intel.com>
Signed-off-by: Gurunath S <gurunath.s@intel.com>
Signed-off-by: Gurunath S <gurunath.s@intel.com>
… base library and also updated modules version issues.

Signed-off-by: Ubuntu <ubuntu@ip-172-31-1-234.us-west-2.compute.internal>
Signed-off-by: Gurunath S <gurunath.s@intel.com>
@Copilot Copilot AI review requested due to automatic review settings April 17, 2025 05:42
Copy link

github-actions bot commented Apr 17, 2025

Dependency Review

The following issues were found:
  • ✅ 0 vulnerable package(s)
  • ✅ 0 package(s) with incompatible licenses
  • ✅ 0 package(s) with invalid SPDX license definitions
  • ⚠️ 1 package(s) with unknown licenses.
See the Details below.

License Issues

ProductivitySuite/ui/react/package.json

PackageVersionLicenseIssue Type
notistack^3.0.2NullUnknown License

Scanned Files

  • ProductivitySuite/ui/react/package.json

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 144 out of 153 changed files in this pull request and generated 1 comment.

Files not reviewed (9)
  • ProductivitySuite/docker_compose/intel/cpu/xeon/set_env.sh: Language not supported
  • ProductivitySuite/ui/react/.env.production: Language not supported
  • ProductivitySuite/ui/react/index.html: Language not supported
  • ProductivitySuite/ui/react/package.json: Language not supported
  • ProductivitySuite/ui/react/public/manifest.json: Language not supported
  • ProductivitySuite/ui/react/public/model_configs.json: Language not supported
  • ProductivitySuite/ui/react/public/robots.txt: Language not supported
  • ProductivitySuite/ui/react/src/App.scss: Language not supported
  • ProductivitySuite/ui/react/src/components/Chat_Assistant/ChatAssistant.module.scss: Language not supported
Comments suppressed due to low confidence (1)

ProductivitySuite/docker_compose/intel/cpu/xeon/compose.yaml:200

  • The dependency reference has been updated from 'llm' to 'llm_codegen', but there is no corresponding 'llm_codegen' service defined in the configuration. Please double-check the service naming for consistency.
      - llm_codegen

pre-commit-ci bot and others added 18 commits April 17, 2025 05:43
Signed-off-by: Gurunath S <gurunath.s@intel.com>
Signed-off-by: Gurunath S <gurunath.s@intel.com>
Signed-off-by: Gurunath S <gurunath.s@intel.com>
Signed-off-by: Gurunath S <gurunath.s@intel.com>
Signed-off-by: Gurunath S <gurunath.s@intel.com>
Signed-off-by: Gurunath S <gurunath.s@intel.com>
… images

Signed-off-by: Gurunath S <gurunath.s@intel.com>
Signed-off-by: Gurunath S <gurunath.s@intel.com>
Signed-off-by: Gurunath S <gurunath.s@intel.com>
Signed-off-by: Gurunath S <gurunath.s@intel.com>
@chensuyue chensuyue added this to the v1.3 milestone Apr 17, 2025
@chensuyue chensuyue added this to OPEA Apr 17, 2025
@chensuyue chensuyue moved this to In progress in OPEA Apr 17, 2025
Signed-off-by: Gurunath S <gurunath.s@intel.com>
Signed-off-by: Gurunath S <gurunath.s@intel.com>
Signed-off-by: Gurunath S <gurunath.s@intel.com>
Signed-off-by: Gurunath S <gurunath.s@intel.com>
@joshuayao joshuayao moved this from In progress to In review in OPEA Apr 18, 2025
@chensuyue
Copy link
Collaborator

chensuyue commented Apr 21, 2025

I can reproduce the issue on my local env, so it's not a test env issue.
https://github.com/opea-project/GenAIExamples/actions/runs/14518876160/job/40769283779?pr=1834
image

Have you run with HUGGINGFACEHUB_API_TOKEN=xxx bash test_compose_on_xeon.sh? I guess the issue may caused by the gap between test scripts parameters and set_env.sh. Suggest to do the same change like this, 8d421b7#diff-ceb7170520623310c6b0ce142edb85cb505ed2561311082300eb73f3d01d8b8b

@lvliang-intel lvliang-intel merged commit ea9d444 into opea-project:main Apr 21, 2025
15 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in OPEA Apr 21, 2025
yongfengdu pushed a commit to yongfengdu/GenAIExamples that referenced this pull request May 8, 2025
Signed-off-by: Gurunath S <gurunath.s@intel.com>
chyundunovDatamonsters pushed a commit to chyundunovDatamonsters/OPEA-GenAIExamples that referenced this pull request May 16, 2025
Signed-off-by: Gurunath S <gurunath.s@intel.com>
Signed-off-by: Chingis Yundunov <c.yundunov@datamonsters.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants