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

Add official demo in Google Colab #184

Merged
merged 4 commits into from
Sep 20, 2022
Merged

Add official demo in Google Colab #184

merged 4 commits into from
Sep 20, 2022

Conversation

DiegoFernandezC
Copy link
Member

This PR introduces a Colab demo similar to the demo running on the Hugging Face Spaces.

When this PR is approved I change the GitHub links to the master brach.

demos/colab/colab_demo.ipynb Outdated Show resolved Hide resolved
import tempfile

import numpy as np
from distance_function import euclidean_distance, iou
Copy link
Collaborator

Choose a reason for hiding this comment

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

this module is available under norfair.distances no need to repeat it in the demo

Copy link
Collaborator

Choose a reason for hiding this comment

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

but be careful with the distance threshold for iou, the implementation provided by norfair is bound to [0,1] where 1 means that the boxes don't touch and 0 means the boxes are the same

Copy link
Member Author

Choose a reason for hiding this comment

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

I changed these distance functions to create_normalized_mean_euclidean_distance like you suggest me on the HF demo.

demos/colab/distance_function.py Outdated Show resolved Hide resolved
Copy link
Collaborator

@javiber javiber left a comment

Choose a reason for hiding this comment

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

a couple more questions

demos/colab/requirements.txt Outdated Show resolved Hide resolved
demos/colab/colab_demo.ipynb Outdated Show resolved Hide resolved
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@DiegoFernandezC DiegoFernandezC force-pushed the chore/colab-demo branch 2 times, most recently from 903445a to d351b6e Compare September 20, 2022 15:19
Copy link
Collaborator

@javiber javiber left a comment

Choose a reason for hiding this comment

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

LGTM

@dekked dekked changed the title chore: add colab demo Add official demo in Google Colab Sep 20, 2022
@DiegoFernandezC DiegoFernandezC merged commit 0f51440 into master Sep 20, 2022
@DiegoFernandezC DiegoFernandezC deleted the chore/colab-demo branch September 20, 2022 17:14
@dekked dekked mentioned this pull request Sep 21, 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.

2 participants