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

Add support for running CalculiX using WSL #9

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

FranseFrikandel
Copy link

Allows CalculiX in WSL to be used instead of a native install within Windows.

Added this feature since it was very handy for me. Running this package fully in WSL wasen't playing nicely with the GUI of gmsh. It constantly kept hanging whenever I tried to use it. However CalculiX binaries that are available online for Windows are generally fairly outdated, and Linux allows far easier installing of CalculiX.

Using this feature, it's possible to run your python scripts in Windows, and have the package call WSL for running CalculiX.

To enable it, simply set Simulation.USE_WSL to True. It will call "wsl.exe ccx" for CalculiX.

Would love to hear if you think this would be an interesting addition to the package.

Definetly still needs some work. Converting between WSL and Windows paths is a bit of a hack at the moment, and I think it won't work for local paths, as I'm using absolute paths for everything for now. Also the way USE_WSL is set might need some helper functions, since that appears to mostly be convention within the package so far.

@FranseFrikandel
Copy link
Author

Turns out PrePoMax does actually include the latest version of CalculiX. It also links to the following URL for a download of the latest CalculiX version: http://www.dhondt.de/calculix_2.21_4win.zip

For whatever reason, this URL doesnt appear to be mentioned anywhere on the website itself, so it appeared only older versions of CalculiX where available.

This makes this PR not very useful anymore - Using CalculiX natively is easier and doesn't force you to use an older version.
This PR also still has the issue that setting core count does not work.

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.

1 participant