Skip to content

Conversation

@Lestropie
Copy link
Member

@Lestropie Lestropie commented Mar 30, 2022

Changes over and above #2370. Using that as the base branch in order to show differences, but will change base branch to master after that is merged.

3af9f6f contains two fixes:

  • Singularity was refusing to run mrview due to an error I'd never seen previously. Don't know whether it is related to a change to the base container OS, or a change to the host system (which is the same one I first generated the container definition files); planning to also test on a different host system before merging.

  • Building the Singularity container via the DockerHub image resulted in mrview not showing icons; yet executing mrview via Docker did show icons. Can't claim to know how that works, but it nevertheless looks like a missing SVG dependency.
    Edit: In addition to no icons, the following terminal messages were also generated in this use case:

    Qt: Session management error: Authentication Rejected, reason : MIT-MAGIC-COOKIE-1 
    authentication rejected
    QCursor: Cannot create bitmap cursor; invalid bitmap(s)
    

I am still chasing issues with Docker running mrview. This morning it was giving me a bunch of error messages but nevertheless executing, whereas now this evening it's not working at all:

No protocol specified
qt.qpa.xcb: could not connect to display :1
qt.qpa.plugin: Could not load the Qt platform plugin "xcb" in "" even though it was found.
This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.

Available platform plugins are: eglfs, linuxfb, minimal, minimalegl, offscreen, vnc, xcb.

Will continue chasing over the next few days.

- Dockerfile: Install runtime SVG support.
- Singularity: Install dbus and execute dbus-uuidgen as this was observed to be necessary to permit execution of mrview on an up-to-date Arch Linux.
- Update documentation with suggestion if mrview does not work on Singularity.
@Lestropie Lestropie added this to the 3.0.4 milestone Mar 30, 2022
@Lestropie Lestropie requested a review from a team March 30, 2022 07:29
@Lestropie Lestropie self-assigned this Mar 30, 2022
@Lestropie
Copy link
Member Author

Error I'm currently getting when executin mrview through Docker (though seems to function fine):

QStandardPaths: XDG_RUNTIME_DIR not set, defaulting to '/tmp/runtime-root'
libGL error: MESA-LOADER: failed to retrieve device information
libGL error: Version 4 or later of flush extension not found
libGL error: failed to load driver: i915
libGL error: failed to open /dev/dri/card0: No such file or directory
libGL error: failed to load driver: i965

@Lestropie
Copy link
Member Author

Lestropie commented Apr 1, 2022

Okay, cb64041 seems to resolve residual issues I was having running mrview through Docker on my Arch Linux laptop. Switching from draft to PR, though currently targets the #2370 branch, so can either merge this then #2370, or change target branch here to master and close #2370 without merging, or merge #2370 and then change the destination branch here to master.

Would appreciate testing of mrview on a wider range of host systems from:

  • Docker build from Dockerfile recipe file
  • Singularity build from Singularity recipe file
  • Singularity build from docker://mrtrix3/mrtrix3:debug_3.0.x

@Lestropie Lestropie marked this pull request as ready for review April 1, 2022 04:32
Prevents the following error:
"QStandardPaths: error creating runtime directory /run/user/1000 (No such file or directory)"
@Lestropie
Copy link
Member Author

Given that neither this nor #2370 have been reviewed, and there is no logical distinction between the two, I'm going to merge this into #2370, and have that as a single review of container modifications (in addition to a single set of images to test, particularly for mrview support).

@Lestropie Lestropie merged commit 749bd65 into containers_fixes Apr 6, 2022
@Lestropie Lestropie deleted the containers_fixes_additional branch April 6, 2022 08:44
@Lestropie Lestropie added the container Docker or Singularity containers label Apr 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug container Docker or Singularity containers gui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants