NatLabRockies / NatLabRockies/r2x-cli
bug: only python version 3.12 works
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 4
- Forks
- 2
- Avg merge
- 1d 4h
- Merged PRs (30d)
- 12
Description
Currently, the CI/CD only packages the python 3.12 dynamic library. In order to support the multiple python versions supported in r2x-core we need to handle multiple python versions of the dynamic library. We do need at least one dylib included in the default installation for the cli to work at all.
One solution could be to have uv add additional cdylibs to the r2x binary folder when a user switches versions, or if for some reason, the users default python version is different than the client default python version.
A less optimal solution would be to package all python cdylib versions as part of the CI/CD pipeline, but this would mean a larger binary archive to install initially.
UPDATE:
We will need to compile separate binaries for each version of python supported in r2x core. When building with PYO3, a hard link is created to the version specific shared object library, so we can't just switch between versions.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
No files or tests are named. Start by inspecting the CI/CD configuration and the PyO3 build setup for r2x-cli, then trace how r2x-core-supported Python versions are selected and how the default installation finds its dylib. Done means separate binaries are built for supported Python versions, with at least one dylib available in the default CLI installation.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python, rust
- Domain
- build-system, ci-cd, cli
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100