-
Notifications
You must be signed in to change notification settings - Fork 24
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
Improve case_generation in example code Class to Support Linux Compatibility #27
Comments
Thanks for your suggestion. Is this for python code in the |
thanks for the reply, I pulled the code from main branch, also I meant something in the python openfast_toolbox, example:
|
Thanks for replying. This issue should be posted on the toolbox rather than here. |
I transferred this issue from OF. |
Hi Mr Thedin, this task for Fast.FARM for CI/CD? Task: “It would be great to make the examples runnable as unit tests. Right now the user needs to wait until the low-res boxes are done before moving forward with the setup (if TurbSim inflow)” |
Improve
case_generation
Class to Support Linux CompatibilityDescription
The
case_generation
helper class currently type-checks only for.exe
files when postprocessing and executing module binaries. This creates compatibility issues on Linux systems, as it assumes the binaries are Windows executables.This limitation forces Python/Linux users to modify the code to adapt it for their environment. To improve cross-platform compatibility, it would be helpful to revise the implementation to use the Python
subprocess
library, which can handle executable files in a platform-independent manner.Proposed Solution
case_generation
class to remove hardcoded type checks for.exe
files.subprocess
library to execute binaries, ensuring compatibility across both Windows and Linux platforms.This update would streamline usage for Linux users and reduce the need for manual code changes.
The text was updated successfully, but these errors were encountered: