Skip to content

Upgrade to python 3.11#378

Merged
mogres merged 2 commits intomainfrom
feature/python-3.11
Sep 9, 2025
Merged

Upgrade to python 3.11#378
mogres merged 2 commits intomainfrom
feature/python-3.11

Conversation

@mogres
Copy link
Copy Markdown
Collaborator

@mogres mogres commented Sep 3, 2025

Problem

Updates the python version to 3.11, since python 3.9 will be deprecated in October 2025

Solution

Added support for python 3.11 and changed the build backend to use uv for faster installs.

Type of change

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • This change requires updated or new tests

Change summary:

  • Update pyproject.toml to use python 3.11
  • Use uv as build backend
  • Update README.md to provide installation instructions
  • Updated the pull request and issue templates
  • Update the github actions to use uv

Steps to Verify:

  1. Install uv
curl -LsSf https://astral.sh/uv/install.sh | sh
  1. Install cellPACK with python 3.11
uv sync
  1. Activate environment
source .venv/bin/activate
  1. Pack a test recipe
pack -r cellpack/tests/recipes/v2/test_spheres.json -c cellpack/tests/packing-configs/test_config.json

You can also run commands using uv run COMMAND without activating the environment.

@mogres mogres requested review from meganrm and rugeli and removed request for rugeli September 3, 2025 20:30
* Update pyproject.toml to use python 3.11
* Update github actions to use uv instead of pdm
@mogres mogres force-pushed the feature/python-3.11 branch from 3056d35 to 7120961 Compare September 3, 2025 20:56
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Sep 3, 2025

Packing analysis report

Analysis for packing results located at cellpack/tests/outputs/test_spheres/spheresSST

Ingredient name Encapsulating radius Average number packed
ext_A 25 236.0

Packing image

Packing image

Distance analysis

Expected minimum distance: 50.00
Actual minimum distance: 50.01

Ingredient key Pairwise distance distribution
ext_A Distance distribution ext_A

@mogres mogres requested review from ascibisz and jcass11 September 4, 2025 21:24
Copy link
Copy Markdown
Collaborator

@rugeli rugeli left a comment

Choose a reason for hiding this comment

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

Worked fine on my end, uv is pretty handy indeed!

@mogres
Copy link
Copy Markdown
Collaborator Author

mogres commented Sep 5, 2025

Worked fine on my end, uv is pretty handy indeed!

Thanks @rugeli! Do we need to do any testing for the cellPACK client? Or would this work regardless.

@rugeli
Copy link
Copy Markdown
Collaborator

rugeli commented Sep 8, 2025

Worked fine on my end, uv is pretty handy indeed!

Thanks @rugeli! Do we need to do any testing for the cellPACK client? Or would this work regardless.

@mogres feel free to merge it! Alli and I can deal with any client issues that come up and update docker files.

@mogres mogres merged commit 15d7eb4 into main Sep 9, 2025
7 checks passed
@mogres mogres deleted the feature/python-3.11 branch September 9, 2025 18:04
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