Skip to content

updated for pyqt6#103

Merged
pariterre merged 4 commits into
pyomeca:masterfrom
pariterre:master
Sep 12, 2025
Merged

updated for pyqt6#103
pariterre merged 4 commits into
pyomeca:masterfrom
pariterre:master

Conversation

@pariterre

@pariterre pariterre commented Sep 11, 2025

Copy link
Copy Markdown
Member

This change is Reviewable

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the codebase to use PyQt6 instead of PyQt5, modernizing the Qt framework dependency and updating all related imports and API calls.

  • Updates PyQt dependency from PyQt5 to PyQt6 (>=6) in environment configuration
  • Migrates all PyQt5 imports to PyQt6 equivalents across visualization, analysis, and UI components
  • Updates Qt API calls to use PyQt6's enum-based approach for constants and properties

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
environment.yml Updates PyQt dependency specification to require version 6 or higher
bioviz/viz.py Migrates PyQt5 imports to PyQt6 and updates Qt constant usage to enum-based syntax
bioviz/qt_ui/rectangle_on_slider.py Updates PyQt imports and color constants to PyQt6 equivalents
bioviz/biorbd_vtk.py Changes PyQt5 imports to PyQt6 for QtWidgets and QtGui modules
bioviz/analyses/muscle_analyses.py Comprehensive refactor with PyQt6 migration and enhanced muscle analysis functionality
bioviz/analyses/ligament_analyses.py Updates PyQt imports and frame shape constant usage for PyQt6 compatibility
bioviz/analyses/c3d_editor_analyses.py Migrates PyQt5 imports and alignment constants to PyQt6 enum-based syntax

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread bioviz/analyses/muscle_analyses.py Outdated
Comment thread bioviz/analyses/muscle_analyses.py Outdated
Comment thread bioviz/analyses/muscle_analyses.py Outdated
pariterre and others added 2 commits September 12, 2025 09:17
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@pariterre
pariterre merged commit 9b2be03 into pyomeca:master Sep 12, 2025
3 of 4 checks passed
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