edgetpu-yolo icon indicating copy to clipboard operation
edgetpu-yolo copied to clipboard

update to last version

Open bogdannedelcu opened this issue 7 months ago • 1 comments

Hello @jveitchmichaelis i just tested your code on an EdgeTPU with a model exported from https://hub.ultralytics.com/models/VNKIIDQ0ngjulGQyQnSg

The code in this repository needs to implement a small change in order to reflect the new structure of the last layer of the model and something related to uint8 vs int8. I forked your repository and added some changes to make it workhere: https://github.com/bogdannedelcu/edgetpu-yolo

I believe your repository should be up to date with this new change, so if you have the time to port these changes or if not let me fill a PR and merge it... how you prefer...

https://github.com/bogdannedelcu/edgetpu-yolo/commit/9515e32b62bed357f487179f39ddafc8ed0f51fd#diff-faf94b021d582c4bb814dad7bbd0bda32cabb36cb0d01f2e8d7eef9317d6cf5d files are nms.py and edgetpumodel.py.

Best regards, Bogdan

bogdannedelcu avatar Nov 19 '23 18:11 bogdannedelcu

PR would be great, thanks!

It would be good to keep back-compatibility though. Can we infer what offset to use (4 vs 5) - I assume so?

On Mon, 20 Nov 2023 at 07:05, bogdannedelcu @.***> wrote:

Hello @jveitchmichaelis https://github.com/jveitchmichaelis i just tested your code on an EdgeTPU with a model exported from https://hub.ultralytics.com/models/VNKIIDQ0ngjulGQyQnSg

The code in this repository needs to implement a small change in order to reflect the new structure of the last layer of the model and something related to uint8 vs int8. I forked your repository and added some changes to make it workhere: https://github.com/bogdannedelcu/edgetpu-yolo

I believe your repository should be up to date with this new change, so if you have the time to port these changes or if not let me fill a PR and merge it... how you prefer...

@.*** #diff-faf94b021d582c4bb814dad7bbd0bda32cabb36cb0d01f2e8d7eef9317d6cf5d https://github.com/bogdannedelcu/edgetpu-yolo/commit/9515e32b62bed357f487179f39ddafc8ed0f51fd#diff-faf94b021d582c4bb814dad7bbd0bda32cabb36cb0d01f2e8d7eef9317d6cf5d files are nms.py and edgetpumodel.py.

Best regards, Bogdan

— Reply to this email directly, view it on GitHub https://github.com/jveitchmichaelis/edgetpu-yolo/issues/13, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAYDMJ6WGVTOFMK4HD5IN3DYFJDAPAVCNFSM6AAAAAA7R4WR6OVHI2DSMVQWIX3LMV43ASLTON2WKOZSGAYDAOJWG4ZDGNQ . You are receiving this because you were mentioned.Message ID: @.***>

jveitchmichaelis avatar Nov 19 '23 18:11 jveitchmichaelis