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
sioyek 2.0: Computer crashes when trying to jump to another chapter #539
Comments
Can't reproduce the issue. I even tested it on a linux system with i3 and with a 4000 page document with more than 10000 table of contents entries. Can you share one of the documents which is causing you problems? |
Unfortunately could not reproduce even with the documents. Why do you think i3 might be the issue, does the problem happen with other WMs? |
Thanks for your effort! Actually, the AUR package might be at fault. Today I tried the AppImage and the table of content functionality works just fine. In fact, even the user interface looks changed (it has a nice-looking bar in the beginning). I'm not sure if this is the result of some setting of the AppImage or if that is the new standard for Sioyek 2.0 (it would make me wonder why my AUR built doesn't have it). I think I'll work with Appimages for the time being. Incidentally (sorry for issue hijack), why is Edit: Thinking about this, could it be that a database file was corrupted or somehow incompatible to how Sioyek 2.0 functions? The AppImage doesn't load any of the databases I had for Sioyek before the 2.0 release. This would also fit with the SQL error cited above. |
The appimage should also be using
It could be corruption, but not an incompatibility, because we have not changed the database.
How do you try to use the previous database file with the new appimage (and what error do you get?) |
I also experienced some issues with 2.0.0 (using bspwm). Various documents didn't load and would only give me a blank page, TOC didn't work and Sioyek randomly crashed, often alongside the error message |
It might be a good idea to leave a comment in the AUR package. I can't do it myself because apparently registering in the website requires a captcha which involves pasting a terminal commands which requires I genuinely want to know what type of crack archlinux developers are smoking to think that was a good idea. |
I wrote a comment for the AUR package. Until now I thought that AUR was a convenient way to quickly get packages not (yet) included with the core repositories. Well, I guess there is a price to quick convenience. |
Can you post sioyek's stdout when you open a file? |
I'm the maintainer of that AUR package. If anyone figures out what I need to change in the PKGBUILD, please let me know. |
Oh, the AppImage does load the '~/.config
I defined the Incidentally, do you prefer In any case, thanks a lot for your help! I'm happy with the AppImage now. |
I think we use |
The issues experienced here might be indicative of a more general problem with Pacman. I don't understand these things well enough to assess the scope of the problem, but I thought I would mention this as (part of) the reason why the AUR package is acting up (the PKGBUILD file might be fine). In case you are interested, the problem is discussed in a recent video from YouTube's DistroTube channel. I think it's fair to say that Arch users should revert to the AppImage, at least for the time being. |
I am wondering whether this is caused by the mupdf version on arch linux being different than that expected by sioyek. Currently the arch linux's mupdf library is at version 1.21.1 while sioyek uses mupdf 1.20.0. |
I use the AUR version, and have the same issue. |
@ahrm Why not to change page fetching from TOC as regular gg works? I have such problems (I use Arch btw), but page jumping works perfectly. Is it hard to make such fix? |
I think the problem is that the mupdf function that parses table of content entries takes different number of arguments in the mupdf used in sioyek vs the mupdf linked in archlinux. Which creates nasty bugs and crashes when it is called. So the fix is not as simple as you are suggesting. You either have to link the correct version of mupdf or create a patch for sioyek or use for example this patch: #831 . (it will be included in the next version of sioyek) |
I installed
I don't know if it breaks TOC function or not, but latest version of PKGBUILD was pushed 10 hours ago and should be up-to-date. Maybe kind maintainers may help with it? @goggle @hrdl-github |
The line that mentions |
@hrdl-github Sadly, but even after full system upgrade and rebuilding |
Could you share one of the documents that causes this and possibly the output of |
PDF: https://annas-archive.gs/md5/6834d7936fce8c766a6fb12b175cb644 (but basically any PDF with TOC)
|
With mupdf 1.23.4 (I haven't tested other versions) this document creates TOC nodes that have intra-page offsets |
Probably 0 for offset_x and the |
@hrdl-github patch from your PR works! Thank you. Finally the last piece of my frustration went away. |
First of all, thanks once more for the amazing work you are doing on Sioyek! It's getting better and better and the new 2.0 release brings some cool new features and changes!
I'm experiencing some strange issue, though. When I open the table of content (t) and choose a chapter to jump to RAM usage suddenly goes through the roof. The computer becomes mostly unusable and I have to reboot. This happens with every PDF file I tried. On the command line I saw that
no such column: nan
is the last message.This is my
prefs_user.config
:I'm using the i3 window manager and suspect it might have something to do with it. Does anyone have an idea what might cause the issue?
The text was updated successfully, but these errors were encountered: