Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Summarizer app fails after uploading txt/pdf/md files #802

Closed
amisskii opened this issue Oct 16, 2024 · 0 comments · Fixed by #804
Closed

Summarizer app fails after uploading txt/pdf/md files #802

amisskii opened this issue Oct 16, 2024 · 0 comments · Fixed by #804

Comments

@amisskii
Copy link

amisskii commented Oct 16, 2024

Bug description

The Summarizer application fails after uploading .txt/.pdf/.md documents.

version: 1.3.1

Image

Operating system

Fedora 40, Windows 10 GPU enabled

Installation Method

from ghcr.io/containers/podman-desktop-extension-ai-lab container image

Version

next (development version)

Steps to reproduce

  • Install summarizer demo through recipe catalog
  • Verify summarizer app is running in AI Apps section
  • Open application using Open AI App icon
  • Trigger summarize on ai-studio README.md
    Wait for summary to be generated

Relevant log output

File "/opt/app-root/lib64/python3.11/site-packages/streamlit/runtime/scriptrunner/script_runner.py", line 600, in _run_script
exec(code, module.dict)
File "/summarizer/summarizer.py", line 120, in
chunks = chunk_text(text)
^^^^^^^^^^^^^^^^
File "/summarizer/summarizer.py", line 62, in chunk_text
count = json.loads(count)["count"]
~~~~~~~~~~~~~~~~~^^^^^^^^^

Additional context

No response

@jeffmaury jeffmaury transferred this issue from containers/podman-desktop-extension-ai-lab Oct 16, 2024
jeffmaury added a commit to jeffmaury/ai-lab-recipes that referenced this issue Oct 16, 2024
jeffmaury added a commit to jeffmaury/ai-lab-recipes that referenced this issue Oct 24, 2024
Fixes containers#802

Signed-off-by: Jeff MAURY <jmaury@redhat.com>
rhatdan added a commit that referenced this issue Oct 28, 2024
fix: update summarizer app to use correct url for token count
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant