-
Notifications
You must be signed in to change notification settings - Fork 31
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
update to last version #13
Comments
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
<bogdannedelcu@9515e32#diff-faf94b021d582c4bb814dad7bbd0bda32cabb36cb0d01f2e8d7eef9317d6cf5d>
files are nms.py and edgetpumodel.py.
Best regards,
Bogdan
—
Reply to this email directly, view it on GitHub
<#13>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAYDMJ6WGVTOFMK4HD5IN3DYFJDAPAVCNFSM6AAAAAA7R4WR6OVHI2DSMVQWIX3LMV43ASLTON2WKOZSGAYDAOJWG4ZDGNQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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...
bogdannedelcu@9515e32#diff-faf94b021d582c4bb814dad7bbd0bda32cabb36cb0d01f2e8d7eef9317d6cf5d
files are nms.py and edgetpumodel.py.
Best regards,
Bogdan
The text was updated successfully, but these errors were encountered: