ragflow icon indicating copy to clipboard operation
ragflow copied to clipboard

Feature/excel export fix

Open Shivamjohri247 opened this issue 1 month ago • 3 comments

PR details

feat: Add Excel export support and fix variable reference regex Changes:

  • Add Excel export output format option to Message component
  • Apply nest_asyncio patch to handle nested event loops
  • Fix async generator iteration in canvas_app.py debug endpoint
  • Add underscore support in variable reference regex pattern

What problem does this PR solve?

Type of change

  • [x] New Feature (non-breaking change which adds functionality)

Shivamjohri247 avatar Dec 11 '25 18:12 Shivamjohri247

@ZhenhangTung i've added changes for excel export as well as excel processor. Please review and do let me know any modifications are needed.

Shivamjohri247 avatar Dec 11 '25 18:12 Shivamjohri247

Appreciations! Could you share us with screenshot GIF or something?

KevinHuSh avatar Dec 12 '25 01:12 KevinHuSh

@KevinHuSh @ZhenhangTung please find the attached slides and the reference for replications on your end. Also attaching the agent yaml for reference as well newsheet.xlsx Untitled spreadsheet 2.xlsx PR 11914 slides.pptx test-agent1.json

Shivamjohri247 avatar Dec 12 '25 07:12 Shivamjohri247

@KevinHuSh @ZhenhangTung just wanted to check back if you got a chance to review this :)

Shivamjohri247 avatar Dec 13 '25 17:12 Shivamjohri247

The screenshot GIF is the most efficient way to demostrate what is this PR all about.

KevinHuSh avatar Dec 15 '25 01:12 KevinHuSh

@KevinHuSh @ZhenhangTung please find it here walkthrough

Shivamjohri247 avatar Dec 15 '25 07:12 Shivamjohri247

@Magicbook1108 this was the question asked in the slides too. I'll add the code changes and push back asap :)

Shivamjohri247 avatar Dec 15 '25 09:12 Shivamjohri247

@Magicbook1108 please find the implementation changes here PR_11914_V2_walkthrough

Shivamjohri247 avatar Dec 15 '25 12:12 Shivamjohri247