pasbi

Results 31 comments of pasbi

@asmorkalov I totally understand your decision. However, what about my proposal: having a function `size()` instead? It wouldn't increase the size of the struct and avoid value duplication. @srinath2468 maybe...

Seems like an older nvcc doesn't like `[[nodiscard]]`. We have two options: 1. introduce a conditional `[[nodiscard]]` macro proxy (I haven't found one in OpenCV) 2. remove `[[nodiscard]]` Which one...

Sorry, maybe my comment was misleading. The PR already has `CV_NODISCARD_STD`, but apparently it decays to `[[nodiscard]]` on old NVCC even though it's not supported (?). Maybe we need to...

According to [cppreference.com compiler support table](https://en.cppreference.com/w/cpp/compiler_support/17), `nvcc` supports `[[nodiscard]]` from version 11. I cannot test that. I have only nvcc >12 at hand and `[[nodiscard]]` works there as expected. I...

Thanks for your reply. > It is possible, indeed, you can separate the creation of the package, containing only the build, and then move the testing (functional or e2e, that...

> I understood that you already had completely separate CI jobs for this, so I thought you had highly decoupled tasks, so having the packaging in the "build" task and...

I think it's better to reopen than creating a new issue because it's very related. @memsharded You said > It is not really necessary to install python things to make...

With "install", I mean what `cmake --install ...` or `cmake --build ... --target install` usually does (without conan) or `cmake.install()` in a `conanfile.py`. There is no need for a virtual...

The problem was solved using workarounds (i.a., applying https://github.com/pybind/pybind11/issues/415#issuecomment-247856419, copying some stuff around manually and joining building and testing into one CI-job). That is, `cmake.install()` is in `package()` and the...

Swapping the quotes did the trick, thanks! `'"(Util|Test).*"'` Though, I don't understand what's happening. The outer single quotes denote the python string, so the actual argument is just `"(Util|Test).*"`. I'm...