Skip to content

Conversation

@HellAholic
Copy link
Contributor

This pull request updates the version of the cpython dependency in the conanfile.py to ensure the project uses the latest stable release.

Dependency updates:

  • Updated the cpython requirement from version 3.12.2 to 3.12.7 in the requirements method of conanfile.py.

CURA-12814

Bump versions for Qt (6.9.1), PyQt6 (6.9.1), cryptography (46.0.1), scipy (1.16.2), cffi (2.0.0), and add numpy (1.26.4) to pip_requirements_core. This ensures compatibility with recent upstream changes and improves security and stability.
@github-actions
Copy link

github-actions bot commented Nov 24, 2025

Test Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit 6c6d7bf. ± Comparison against base commit 4cbced7.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@wawanbreton wawanbreton left a comment

Choose a reason for hiding this comment

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

Some minor remarks, however with this change we completely lose the optimized version of numpy on Windows. I see 2 options:

  • Test if that actually makes a noticeable change when using Cura on Windows (e.g. performance while painting). Hopefully this is not really required.
  • Add a windows sub-version for ARM that does not use the optimized numpy

Bumped Qt, PyQt6, and PyQt6-Qt6 versions from 6.9.1 to 6.10.0 in extra_dependencies and pip_requirements_core. Updated associated hashes for PyQt6 and PyQt6-Qt6. Also moved numpy hashes to the correct section for consistency.
@HellAholic
Copy link
Contributor Author

The package for the numpy + mkl is only effective for Intel cpu on windows devices, we can make a comparison and if deemed to be not necessary it helps to clean things up. We can use this version as measure for the comparison against the baseline with that package.

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.

3 participants