Skip to content

Conversation

@astrofrog
Copy link
Member

The extras mentioned did not all work

Copy link
Contributor

@dhomeier dhomeier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Covers largely my thoughts on qt6 install options.

Comment on lines 19 to 20
The ``[qt]`` ensures PyQt5 gets installed, but you can also install PyQt6 or
PySide yourself and miss out the ``[qt]``.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
The ``[qt]`` ensures PyQt5 gets installed, but you can also install PyQt6 or
PySide yourself and miss out the ``[qt]``.
pip install PyQt6 glue-qt # Qt6-based front-end
The ``[qt]`` ensures PyQt5 gets installed and set as the front-end,
but you can also use **instead** an already installed PyQt6 or PySide
front-end, or install it along the glue libraries as above (omitting the ``[qt]``).

Perhaps clarifying the different options a bit better here? Or maybe we should link/refer to the qtpy documentation how the default front-end is picked or prescribed setting QT_API?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree we can expand this a bit more - for now I've pushed an update along the lines you suggest, then let's get this PR in so the instructions are correct, then we can do a follow-up with some improvements.

@astrofrog astrofrog force-pushed the update-install-instructions branch 2 times, most recently from 9b7b8b5 to a4d21d8 Compare May 13, 2025 09:35
@astrofrog astrofrog merged commit 25ce608 into glue-viz:main May 13, 2025
4 checks passed
@astrofrog astrofrog added the documentation Improvements or additions to documentation label Aug 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants