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

Fixes some serverless functions (HRNET, IOG, f-BRS) #4944

Merged
merged 5 commits into from
Oct 9, 2022

Conversation

amteusch
Copy link
Contributor

Motivation and context

Fixes #4736, and also includes a fix for HRNET from @IuliuNovac in #4900. Some serverless functions (IOG, f-BRS) were downloading weights from Google Drive, but Google Drive changed and weights were not downloading properly, this should address that. Also updates the yaml files for a few serverless functions that had outdated or incompatible versions of python, opencv, etc. There might be a better way to fix these yamls but this is simply how I got them working and figured it's better than nothing.

How has this been tested?

Tested the serverless functions running locally on Ubuntu 20.04.5. They seemed to work fine, which is better than the current state of the repo, but I'm not sure if there is any performance decrease relative to when they worked before.

Checklist

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.

@amteusch amteusch marked this pull request as draft September 13, 2022 18:43
@amteusch amteusch marked this pull request as ready for review September 13, 2022 18:59
@amteusch
Copy link
Contributor Author

amteusch commented Oct 5, 2022

@yasakova-anastasia @bsekachev I see some of these changes made it in on another pull request, any way we can get this HRNET change too?

@bsekachev
Copy link
Member

@yasakova-anastasia

Could you please check HRNet?

@yasakova-anastasia
Copy link
Contributor

Could you please check HRNet?

Yes, HRNet is not working. These changes solve this problem

@bsekachev
Copy link
Member

@amteusch

Thank you for the contribution! :)

@bsekachev bsekachev merged commit c8c40df into cvat-ai:develop Oct 9, 2022
@nmanovic nmanovic mentioned this pull request Dec 12, 2022
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.

fbrs semi automatic annotation nuclio function starts unhealhy
3 participants