-
Notifications
You must be signed in to change notification settings - Fork 2
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
I noticed that your repo was updated #1
Comments
Hi @psiydown, since I am working on this stuff right now I did some changes to the code and minor modification must be done in order to use the scripts, especially for |
Hi @psiydown, I finally made a clear guide to have everything working! I tested it and just by cloning the repo, running the scripts in |
Hi @StefanoBerti ,Good job, it's a great job! I changed a new rtx3060 graphics card and tested it. It runs well and can reach 16 FPS! There are several questions:
and
You did a very good job, Thank you very much! |
Thank you!
|
Hi @StefanoBerti ,
|
|
Hi @StefanoBerti , |
@psiydown Really? That's interesting! |
Hi @StefanoBerti , |
I tested the original model of metroabs, reduced number of test-time augmentations |
@psiydown It could be that you are making the same mistake that I did few time ago. The video has resolution 1920x1080, which means that it has an aspect ratio of 1920/1080=16:9. If you resize an image with that shape into the shape 640x480, you are changing the aspect ratio from 16:9 to 640/480=4:3. The correct way to preprocess such video is |
@StefanoBerti No, I didn't make the same mistake. I pay great attention to precision and detail. I have scaled the video with the same proportion as the original video, and the excess border is filled with black. |
@psiydown maybe you are calling |
@StefanoBerti According to your tips, I successfully use Onnxrtrunner of Polygraphy to predict with onnx model. However, the predict result is the same as that of the engine, which shows that the accuracy is not reduced by creating the engine. It may be that the accuracy has been reduced when converting the onnx model, or as you said, it is caused by preprocessing or post-processing. Can you test the video I sent to find out the reasons and solutions for the decline in accuracy? Thank you! Because I need to get the absolute pose, I use |
@psiydown What do you mean with "testing the video"? I tried it and it works quite well imo. Anyway I don't use the I think that the fact that you use |
@StefanoBerti This is not the reason, I tested to delete |
@psiydown ok thank you now I got what you mean! Well that pose is very hard to estimate btw, I don't know how the original model could handle it. Since you have tested
|
@StefanoBerti I read the source code of metabs and set the parameter I have no other way, I pay more attention to model accuracy and loading speed than predict speed. Do you have any way to use the original model, reduce the memory occupation and improve the loading speed? I hope you can give me specific tips. Thank you very much! |
@psiydown if these changes in accuracy are fundamentals for your implementation then yes, I think that using the original model works better for you. I have no clue what else is missing or is wrong here. You can use the XLA optimization of TensorFlow to increase the number of FPS to 4.5, in my case it worked. Let me know if you manage to discover how to improve the accuracy! |
@StefanoBerti If I find out how to improve the accuracy, I'll let you know at the first time. Try XLA can indeed improve the predict speed, but the loading speed of the original model is still very slow, Each loading takes up 11g of memory by waiting for a few minutes. I found that the metabs model extracted by your program is relatively small. The extracted original model does not need to be converted onnx, Can it be predict directly? How to realize it? |
@psiydown sorry but I didn't understand your question. There are a lot of preprocessing and postprocessing that I tried to replicate accurately to avoid to use TensorFlow, but if you want to use the original model you need to accept to use TensorFlow. Anyway, you can find how to reduce the GPU memory used by TensorFlow |
Hi @StefanoBerti ,I noticed that your repo was updated, I tested, but the following errors occurred:
What are the functions of the new TRX and LSTM?
Do you have a gtx1060 or gtx1080 graphics card? Can you test its compatibility with your program?
The text was updated successfully, but these errors were encountered: