chrisgrieser / pdf-summarizer-alfred

Get summaries of your PDFs via ChatPDF.
MIT License
16 stars 1 forks source link

[Bug]: the app doesn't retrieve information from chatpdf #2

Closed jhs1965net closed 10 months ago

jhs1965net commented 10 months ago

Bug Description

I only retrieve : [ { \

Relevant Screenshot

Screenshot 2024-01-10 at 13 05 07

To Reproduce

toggle Alfred and by universal key or direct from finder the script begins but just give me : [ { \

in a text file

Debugging Log

[13:10:50.633] Logging Started...
[13:12:17.515] PDF Summarizer[Hotkey] Processing complete
[13:12:17.524] PDF Summarizer[Hotkey] Passing output '(
    "/Users/jameshamilton/Downloads/Harper, K. N. - Armelagos, G. J. 2013, [87545],Genomics, the origins of agriculture, and our changin copy 2.pdf"
)' to Run Script
[13:12:18.113] STDERR: PDF Summarizer[Run Script] curl: (26) Failed to open/read local data from file/application
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100  1325  100  1042  100   283   2854    775 --:--:-- --:--:-- --:--:--  3630
[13:12:18.119] PDF Summarizer[Run Script] Processing complete
[13:12:18.120] PDF Summarizer[Run Script] Passing output '' to Large Type
[13:12:18.121] PDF Summarizer[Run Script] Passing output '' to Run Script

Workflow Configuration

(removed due to API key leaking)

Checklist

chrisgrieser commented 10 months ago

There seems to be a problem with the PDF. Could you attach the PDF, so I can have a look?

jhs1965net commented 10 months ago

Here it is, but I have also the problem with another pdf James On 10 Jan 2024, at 13:34, pseudometa wrote:

There seems to be a problem with the PDF. Could you attach the PDF, so I can have a look?

-- Reply to this email directly or view it on GitHub: https://github.com/chrisgrieser/pdf-summarizer-alfred/issues/2#issuecomment-1884768092 You are receiving this because you authored the thread.

Message ID: @.***>

chrisgrieser commented 10 months ago

there is no attachment. Please do not use email to respond to github issues, since they are quite buggy.

jhs1965net commented 10 months ago

Harper, K. N. - Armelagos, G. J. 2013, [87545],Genomics, the origins of agriculture, and our changin copy 2.pdf

jhs1965net commented 10 months ago

Varian, B. J., Weber, K. T. & Erdman, S. E. 2023, [121406],Oxytocin and the microbiomeComprehensive 5259.pdf

chrisgrieser commented 10 months ago

The issue is that your filenames include ,. Commas are treated as enumeration, so ChatPDF thinks you are trying to upload multiple files, one Varian, another B. J., another Weber, etc. And since those files do not exist, ChatPDF fails.

That's an issue on ChatPDF's side, and unfortunately there is nothing I can do about that on my end. Simply removing the commas from the filenames will fix the issue though.

By the way, there are actually many apps that have problems with commas (or other special characters) in filenames, so it is actually good practice to only have letters, digits, and the -_. characters in your filenames, since this also avoids annoying issues elsewhere.

jhs1965net commented 10 months ago

Chris I really appreciate your time and concern, I am writing a book about evolution, epigenetic and socialisation more than 3000 pdf in bookends and Zotero so I really enjoy using your app and read about all the knowledge you write down. I have also some publications about Adverse childhood experiences and I ran a little team for research in those topics kindly and thanks a lot.

Dr James Hamilton

jhs1965net commented 10 months ago

it worked !!!! is there any possibility to chat with the same pdf?

chrisgrieser commented 10 months ago

Unfortunately not, this workflow is only to get a quick-and-dirty summary.

To chat with the PDF, you can simply use the ChatPDF website?