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

pynmstar failure to compile sub package #7589

Open
varioustoxins opened this issue Sep 20, 2024 · 10 comments
Open

pynmstar failure to compile sub package #7589

varioustoxins opened this issue Sep 20, 2024 · 10 comments
Assignees
Labels
needs-mre Needs more information for reproduction

Comments

@varioustoxins
Copy link

HI
if I do uv tool install nef-pipelines it all appears to work
However, when I run the command nef [the main entry point to my application in nef-pipelines], I get

raise ImportError('Could not import cnmrstar sub-module! Your installation appears to be broken.')

This appears to be because the dependant c file from the package pynmstar.cnmrstar doesn't appear to be compiled properly. I appear to have had similar but different problems as the c component seems to require the latest setup tools to compile. So when using pip I have to do

pip install --upgrade wheel setuptools

and sometimes

pip install --upgrade wheel setuptools backports.tarfile

to get the installation to work, this is on a centos 7 system but I have had this problem elsewhere

this is with the latest uv at time of writing 0.4.13

@varioustoxins
Copy link
Author

this is also discussed here, binary wheels are provided so it shouldn't need to build the c

@charliermarsh
Copy link
Member

Can you share the output of uv tool install with --verbose?

@varioustoxins
Copy link
Author

here you go
output.txt

@charliermarsh
Copy link
Member

So for one: pynmrstar only ships wheels for Python 3.9 and earlier. You're on Python 3.12, so I think it's correct that it's building from source.

@charliermarsh
Copy link
Member

See here (look at cp39 etc.).

@charliermarsh
Copy link
Member

It seems to install ok for me:

❯ nef
Usage: nef [OPTIONS] COMMAND [ARGS]...

Options:
  --install-completion [bash|zsh|fish|powershell|pwsh]
                                  Install completion for the specified shell.
  --show-completion [bash|zsh|fish|powershell|pwsh]
                                  Show completion for the specified shell, to
                                  copy it or customize the installation.
  --help                          Show this message and exit.

Commands:
  chains     - carry out operations on chains
  csv        - read [rdcs]
  deep       - read deep [peaks]
  echidna    - read echidna data [peaks]
  entry      - carry out operations on the nef file entry
  fasta      - read and write fasta sequences
  fit        - carry out fitting operations [alpha]
  frames     - carry out operations on frames in nef files
  globals    - add global options to the pipeline [use save as your last...
  header     - add a header to the stream
  help       - help on the nef pipelines tools and their usage
  loops      - carry out operations on loops in nef frames
  mars       - read and write mars [shifts and sequences]
  modelfree  - write modelfree [relaxation data]
  nmrpipe    - read nmrpipe [peaks shifts & sequencess]
  nmrstar    - read NMR-STAR [sequences & shifts]
  nmrview    - read and write nmrview [peaks, sequences & shifts]
  pales      - read and write pales/dc [rdcs]
  pdbx       - read pdb [sequences]
  peaks      - carry out operations on nef peaks
  rpf        - write rpf shifts
  save       - save the entries in the stream to a file / files or stdout...
  series     - carry out operations on a data series
  shifts     - carry out operations on shifts [average]
  shiftx2    - read shiftx2 data [shifts]
  shifty     - write shifty [shifts]
  simulate   - simulate data
  sink       - read the current stream and don't write anything
  sparky     - read sparky files [shifts]
  stream     - stream a nef file
  talos      - read and write talos files [shifts & restraints]
  test       - run the test suite
  xcamshift  - write xcamshift for xplor [shifts]
  xeasy      - read xeasy files [flya dialect: sequence]
  xplor      - read xplor [sequences, dihedral & distance restraints]

Do you see a failure when running just nef? Or when you run nef within a project, some specific subcommand, etc.?

@charliermarsh charliermarsh added the question Asking for clarification or support label Sep 20, 2024
@charliermarsh charliermarsh self-assigned this Sep 20, 2024
@varioustoxins
Copy link
Author

varioustoxins commented Sep 20, 2024

just running it as a command as you did... it maybe a centos 7 oddity it is quite old, I have tried it on a couple of centos 7 installations, so I assume that its not just a wrinkle on a particular install

@charliermarsh
Copy link
Member

Are you able to put together a reproduction, e.g., with Docker?

@charliermarsh charliermarsh added needs-mre Needs more information for reproduction and removed question Asking for clarification or support labels Sep 25, 2024
@varioustoxins
Copy link
Author

Hi Charlie

I should be able to, but it will take a little while as I am currently on holiday till the 24th and I will need an hour or two to sort something out.

gary

@charliermarsh
Copy link
Member

No problem, thanks for following up. Enjoy your holiday.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-mre Needs more information for reproduction
Projects
None yet
Development

No branches or pull requests

2 participants