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

Allow to not use a pretrained FairMOT backbone #348

Merged
merged 5 commits into from
Dec 2, 2022

Conversation

iliiliiliili
Copy link
Collaborator

This PR fixes the dependency on the external model downloads by not requiring a pretrained backbone if you intend to test using the trained model from the FTP server.

@iliiliiliili iliiliiliili added test sources Run style checks test tools Test the toolkit methods labels Nov 15, 2022
Copy link
Collaborator

@tsampazk tsampazk left a comment

Choose a reason for hiding this comment

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

Thanks you for these changes, i have a minor comment. Other than that they look good!

Copy link
Collaborator

@ad-daniel ad-daniel left a comment

Choose a reason for hiding this comment

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

Thank you, hopefully it will reduce the number of failures

Copy link
Collaborator

@passalis passalis left a comment

Choose a reason for hiding this comment

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

Thank you!

@ad-daniel ad-daniel merged commit 7c5a113 into develop Dec 2, 2022
@ad-daniel ad-daniel deleted the fix-fairmot-model-download branch December 2, 2022 15:17
lucamarchionni pushed a commit to lucamarchionni/opendr that referenced this pull request Jun 10, 2024
* Allow to not use a pretrained fairmot backbone

* Rename load_pretrained_model

Co-authored-by: Kostas Tsampazis <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test sources Run style checks test tools Test the toolkit methods
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants