-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Adding support for rank cython in Windows, fixing pytorch deprecation warnings #412
Conversation
Thanks so much for fixing this long-standing issue for Windows users! One more step to go: can you please test the performance of resnet50 (you can download the weights here https://kaiyangzhou.github.io/deep-person-reid/MODEL_ZOO.html) on Market1501/DukeMTMC-reID using both the evaluation codes before and after the changes? I'd like to see whether the calculation precision will be affected. |
I might be doing something wrong but I can't load the pretrained weights
I've downloaded and I'm getting
Getting the same error when using the |
I also met the UnicodeDecodeError when using resnet50. Hmm, that's weird. Maybe try osnet, which works for me
|
This one worked. please see the attached logs. Only difference (aside from the version) is the Speed, 0.0634 sec/batch vs 0.0638 sec/batch, which I guess is negligible This was run on Ubuntu 18.04 Windows 1.3.5 (Had to lower the batch size since it's a laptop with less memory on the GPU and kept getting pytorch out of memory issues) |
Great! The speed isn't really an issue because you normally get different speeds in different runs. I'll merge this code with the master branch. Thanks! |
Following suggestion here
ansys/pymapdl#14 (comment)
I got the cython to run on Windows and checked on Linux as well
Tested on Windows 64bit & Ubuntu 18.04