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

Update Readme #37

Merged
merged 2 commits into from
Apr 10, 2024
Merged

Update Readme #37

merged 2 commits into from
Apr 10, 2024

Conversation

fsimonis
Copy link
Collaborator

@fsimonis fsimonis commented Apr 9, 2024

This PR updates the Readme and adds a __main__.py to the package to get nuitka working again.

Note that compiling the script with nuitka leads to an error in vtk. So far I couldn't find a way to fix this.

When I run python -m nuitka --follow-imports --python-flag=-m ccx2paraview (same for --standalone) and then run the result I get :

$ ./ccx2paraview.bin
Traceback (most recent call last):
  File "/.../ccx2paraview/ccx2paraview/__main__.py", line 1, in <module>
    from . import cli

  File "/.../ccx2paraview/ccx2paraview/cli.py", line 13, in <module ccx2paraview.cli>
    from .common import Converter

  File "/.../ccx2paraview/ccx2paraview/common.py", line 21, in <module ccx2paraview.common>
    import vtk

  File "/.../venv/lib/python3.11/site-packages/vtk.py", line 47, in <module vtk>
    from vtkmodules.vtkRenderingMatplotlib import *

ImportError: /.../ccx2paraview/venv/lib/python3.11/site-packages/vtkmodules/libvtkPythonInterpreter-9.2.so: undefined symbol: Py_RunMain

@imirzov
Copy link
Collaborator

imirzov commented Apr 10, 2024

Since now we do not need to create binaries at all.

@imirzov imirzov merged commit 32434df into calculix:master Apr 10, 2024
@fsimonis fsimonis deleted the update-readme branch April 10, 2024 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants