Open pkestene opened 1 year ago
Leaving @szellmann to deal with the code itself, but one question: "std::strtok" doesn't sound right !? strtok is a posix C function, it shouldn't even know what a namespace is!?!? Might need to include string.h to get it, but it should never be in a std:: namespace? confused :)
I think you need to either include <string.h>
(C header, fine using strtok
) or either include <cstring>
(which brings back into std namespace the c function, and in that case, the full name is std::strtok
).
I was suggesting to include <cstring>
.
Hi,
here are two minor build issues that seem to be related with compiler version (I'm using gnu 11 on my host, and cuda 12), and those minor issues can be fixed easily
function saturate
I saw that this has already been fixed in upstream owl (October 2022, https://github.com/owl-project/owl/commit/ef3f473c908943928d53f22ef2ba6e0f435b99bb). Currently, updating the owl submodule brings other build issues related to optix version (that be dealt in a separated issue)
the second minor problem is just a missing include, and replacing strtok by std::strtok it builds fine.