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

Can't build tensorrt on Windows x86_64 #858

Closed
devjeonghwan opened this issue Mar 30, 2020 · 6 comments · Fixed by #860
Closed

Can't build tensorrt on Windows x86_64 #858

devjeonghwan opened this issue Mar 30, 2020 · 6 comments · Fixed by #860

Comments

@devjeonghwan
Copy link
Member

bash cppbuild.sh -platform windows-x86_64 -extension "-gpu" install tensorrt

Detected platform "windows-x86_64"
Building for platform "windows-x86_64" with extension "-gpu"
Installing "tensorrt"
/d/javacpp/javacpp-presets/tensorrt/cppbuild /d/javacpp/javacpp-presets
Error: Platform "windows-x86_64" is not supported
/d/javacpp/javacpp-presets

How to build tensorrt for windows x86_64? I think tensorrt's headfile and interface are same on Windows and Linux.

@saudet
Copy link
Member

saudet commented Mar 30, 2020

It's not currently supported, but it might just be a question of adding the right paths to these files for @Platform(value = "windows", ...):
https://github.com/bytedeco/javacpp-presets/blob/master/tensorrt/cppbuild.sh
https://github.com/bytedeco/javacpp-presets/blob/master/tensorrt/src/main/java/org/bytedeco/tensorrt/presets/nvinfer.java

@devjeonghwan
Copy link
Member Author

thx for your reply.

i have one more question.

if i add win tensorrt header file path and win platform string to nvinfer.java and remove the condition statement in cppbuild.sh.
can i build on windows?

@saudet
Copy link
Member

saudet commented Mar 30, 2020

If the cppbuild.sh script fails, the build is going to fail. Please read the following page:
https://github.com/bytedeco/javacpp-presets/wiki/Create-New-Presets

@devjeonghwan
Copy link
Member Author

thank you! i added new windows preset to tensorrt module.

now, i can use tensorrt on windows!

@saudet
Copy link
Member

saudet commented Mar 30, 2020

Please send a pull request with your modifications!

@saudet saudet reopened this Mar 30, 2020
@devjeonghwan
Copy link
Member Author

ok! i will send pr tomorrow(because, i want clean my code)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants