Skip to content
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

Move to QtPy compatibility #1404

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

mrvisscher
Copy link
Collaborator

For a while now we've wanted to move to PySide6. However, because of missing libraries in the Anaconda distribution of that package we were stuck using PySide2. Impending incompatiblity of PySide2 with newer versions of Python and a desire to also publish the AB on PyPI has led us to make an intermediate move using a library called QtPy.

This library serves as an interface between the Activity Browser and any Qt back-end, allowing us to build for PySide2 and PySide6 simultaneously. Users can install a clean Conda version that uses PySide2, but they can also opt to install PySide6 using pip and use that as the backend.

In the feature we hope to move to PySide6 entirely, but as long as that Anaconda distribution is not fully supported, we are going to be offering this intermediate solution.

Checklist

If you have write access (otherwise a maintainer will do this for you):

  • Categorize the PR by setting a good title and adding one of the labels:
    bug, feature, ui, change, documentation, breaking, ci
    as they show up in the changelog.
  • Add a milestone to the PR (and related issues, if any) for the intended release.
  • Request a review from another developer.

@coveralls
Copy link

coveralls commented Nov 8, 2024

Coverage Status

coverage: 54.205% (-0.005%) from 54.21%
when pulling ec928fc on mrvisscher:qtpy-compat
into 9ace464 on LCA-ActivityBrowser:main.

@mrvisscher mrvisscher mentioned this pull request Nov 8, 2024
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants