Closed yetigit closed 1 year ago
So the current version does not contain spdlog? What is the error of keeping that?
@SarahWeiii for some obscure reason the logger update commit introduces uncomprehensible errors and compilation issues. not only is it complicated to make the spdlog dependency work nicely on windows and cmake and vcpkg but also past this there is just nothing helpful to fix the situation in terms of error. the Model
class will stay an undefined type and other weird things.
This is the maximum I could do. if you are ok with this PR erasing the update logger code tell me and I'll fix the conflict of the PR.
Hi I have added an option in CMakeLists.txt for turning off spdlog, could you try if your code works on top of the current version?
thanks, 👍 will try in a day or two
Hi we have updated the code and support the Python version on Windows now.
fix for issue : https://github.com/SarahWeiii/CoACD/issues/9 Here's the PR @SarahWeiii
things to know about it :
before merging please make sure it works on your linux machine