We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Hi @hojae-io and @se-hwan again,
I know I am a bit annoying but when installing cusadi and trying it out, I encountered two minor bugs:
The script to generate the code expects an existing build-folder:
build
cusadi/run_codegen.py
Line 38 in 6efc56c
README.md
The generated CMakeLists.txt contains the line:
CMakeLists.txt
cusadi/src/generateCUDACode.py
Line 17 in 6efc56c
On that remark, one question: Why do you upload the CMakeLists.txt when it is generated either way?
As before, let me know if you want my help on this one
Best, @maxspahn
The text was updated successfully, but these errors were encountered:
Fix issue se-hwan#7
d6075d9
- Add mkdir -p build option in run_codegen.py - Remove deprecated find_package(CUDAToolkit REQUIRED) from generateCUDACode.py - Add build/ and generated CMakeLists.txt file in gitignore
b599169
No branches or pull requests
Hi @hojae-io and @se-hwan again,
I know I am a bit annoying but when installing cusadi and trying it out, I encountered two minor bugs:
The script to generate the code expects an existing
build
-folder:cusadi/run_codegen.py
Line 38 in 6efc56c
You can fix this by either asking the user to create one in the
README.md
, or adding it in the script.The generated
CMakeLists.txt
contains the line:cusadi/src/generateCUDACode.py
Line 17 in 6efc56c
This feature has been deprecated in more recent versions of cmake, see https://cmake.org/cmake/help/v3.20/module/FindCUDA.html. For me it was fine to simply remove that line.
On that remark, one question: Why do you upload the
CMakeLists.txt
when it is generated either way?As before, let me know if you want my help on this one
Best,
@maxspahn
The text was updated successfully, but these errors were encountered: