Skip to content
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

Yolo duplication fix #1713

Merged
merged 78 commits into from
Jan 13, 2025
Merged

Conversation

Sam948-byte
Copy link
Contributor

@Sam948-byte Sam948-byte commented Jan 12, 2025

Somebody wanted a description, so here I am.

This PR fixes an error which caused the discoverModels function to be rerun after each upload of a new model, but without clearing the list of available models. This causes any models that were on the list prior to the import to be duplicated. This PR also makes it so that uploading a model automatically updates the list of available models.

Fix Working on the Wrong Branch
@Sam948-byte Sam948-byte requested a review from a team as a code owner January 12, 2025 06:07
forgot to clean that up
@Sam948-byte
Copy link
Contributor Author

Sam948-byte commented Jan 12, 2025

so it says fix, but it might be broken now. Lemme test. we good now

@Sam948-byte
Copy link
Contributor Author

tested on Chris's opi and it works

Copy link
Contributor

@gerth2 gerth2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

works

@gerth2 gerth2 self-requested a review January 13, 2025 01:48
@gerth2 gerth2 merged commit 966b9e8 into PhotonVision:main Jan 13, 2025
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants