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

Updated dependencys and minimum python version #2

Merged
merged 5 commits into from
Jan 20, 2025
Merged

Updated dependencys and minimum python version #2

merged 5 commits into from
Jan 20, 2025

Conversation

ulvs
Copy link
Member

@ulvs ulvs commented Jan 16, 2025

Updated the qtm-rt and pylsl version to the latest avalible and changed code to mirror the changes in the pacakges.

Changed minimum required Python version to 3.10 and modified asyncIO code and tests due to deprecated functionality.

General code cleanup and formatting.

Tested with Python 3.10, 3.11, 3.12 and 3.13.

All tests passes.

@ulvs ulvs requested review from helmesjo and schoondw January 16, 2025 15:37
Copy link
Contributor

@schoondw schoondw left a comment

Choose a reason for hiding this comment

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

Tested the example

Copy link
Collaborator

@helmesjo helmesjo left a comment

Choose a reason for hiding this comment

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

Can't set this up and test myself atm, but looks like it's mostly formatting changes and a few tweaks for the updated python version requirement (and you have already tested it), so feel free to complete.

@ulvs ulvs merged commit 44945aa into master Jan 20, 2025
@ulvs ulvs deleted the deps_update branch January 20, 2025 13:45
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.

4 participants