Skip to content

feat: integrated LLM Assistant for PDF reports#12

Merged
NeurArk merged 2 commits into
mainfrom
feature/10-llm-assistant
May 7, 2025
Merged

feat: integrated LLM Assistant for PDF reports#12
NeurArk merged 2 commits into
mainfrom
feature/10-llm-assistant

Conversation

@NeurArk

@NeurArk NeurArk commented May 7, 2025

Copy link
Copy Markdown
Owner

Summary

  • Added agent with OpenAI integration to handle natural language requests
  • Enhanced PDF generation tool to handle nested data handling
  • Added special case handling for data from agent in PDF format
  • Fixed display issues with PDF reports including nested data structures
  • Extended test suite for better PDF error handling
  • Improved demo scripts for command-line testing

Test plan

  • All unit tests are passing for PDF generation
  • End-to-end tests verify agent/SQL/PDF integration
  • Agent correctly handles 'create PDF' natural language requests
  • CLI demo script available for manual testing

NeurArk added 2 commits May 7, 2025 16:43
- Added agent with OpenAI integration to handle natural language requests
- Enhanced PDF generation tool to handle nested data
- Added special handling for data from the agent
- Fixed several display issues with PDF reports
- Extended test suite for better error handling
- Improved demo scripts for CLI testing
- Fixed missing whitespace around arithmetic operators in app.py and pdf_tool.py
- Fixed line break positioning for binary operators in tests and pdf_tool.py
- Updated GitHub CI configuration to ignore conflicting rules W503 and W504
@NeurArk NeurArk merged commit 3a0df75 into main May 7, 2025
@NeurArk NeurArk deleted the feature/10-llm-assistant branch May 7, 2025 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant