-
Notifications
You must be signed in to change notification settings - Fork 2
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
thread 'main' panicked #30
Comments
Another one:
Here is the Full Backtrace:
|
If I remember correctly this is due to the library I'm using to cache in markdown format. Could you try to remove all markdown files in your cache and try again? If this issue is due to the cache mode for markdown files, I will have to remove this option for caching in markdown format temporarily to maintain a stable version of bogrep. |
Yes, I am having trouble with the fetching of markdown. With the same source, and repeating fetching, using markdown mode the fetching hangs after a random number of cached files.
… On Nov 6, 2023, at 8:53 PM, quambene ***@***.***> wrote:
If I remember correctly this is due to the library I'm using to cache in markdown format.
Could you try to remove all markdown files in your cache and try again?
If this issue is due to the cache mode for markdown files, I will have to remove this option for caching in markdown format temporarily to maintain a stable version of bogrep.
—
Reply to this email directly, view it on GitHub <#30 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAM6OT2SH4V5RH5C4JSDTDYDGIBHAVCNFSM6AAAAAA673V6VKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJXGE2TGMRTG4>.
You are receiving this because you authored the thread.
|
There also seems to be a memory leak problem. I left the |
I will remove markdown support temporarily in bogrep to fix that issue. |
Fixed on main branch by removing markdown support. FYI, main branch includes changes for the config path on macOS: https://github.com/quambene/bogrep#supported-operating-systems |
Will you bring back markdown support? For the brief time it was working, it had nice output. |
It might be fixed already by filtering out the binary responses. will check later |
@mattmartini The related MR can be found here: #48 (WIP) |
A search panicked the main thread. Possibly a pdf was included in the cache?
Here is the full backtrace:
The text was updated successfully, but these errors were encountered: