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.

Sign up for GitHub

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

Jump to bottom

InstantID notebook #1698

Merged
merged 8 commits into from
Feb 13, 2024
Merged

InstantID notebook #1698

merged 8 commits into from
Feb 13, 2024

Conversation

eaidova
Copy link
Collaborator

@eaidova eaidova commented Feb 8, 2024 •
edited by andrei-kochin
Loading

CVS-131120

@review-notebook-app Review Notebook App
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@eaidova eaidova requested review from a team, apaniukov, itrushkin and aleksandr-mokrov and removed request for a team February 9, 2024 10:46
@eaidova eaidova force-pushed the ea/iinstantid branch 2 times, most recently from eda5cca to 6f2b121 Compare February 9, 2024 12:51
@review-notebook-app Review Notebook App
Copy link

review-notebook-app bot commented Feb 10, 2024 •
edited
Loading

View / edit / reply to this conversation on ReviewNB

aleksandr-mokrov commented on 2024-02-10T01:55:54Z
----------------------------------------------------------------

Here is no a table of contents

@review-notebook-app Review Notebook App
Copy link

review-notebook-app bot commented Feb 10, 2024 •
edited
Loading

View / edit / reply to this conversation on ReviewNB

aleksandr-mokrov commented on 2024-02-10T01:55:54Z
----------------------------------------------------------------

Line #4.    from skimage import transform as trans


ModuleNotFoundError: No module named 'skimage'



eaidova commented on 2024-02-12T10:07:50Z
----------------------------------------------------------------

added to install requirements

@review-notebook-app Review Notebook App
Copy link

review-notebook-app bot commented Feb 10, 2024 •
edited
Loading

View / edit / reply to this conversation on ReviewNB

aleksandr-mokrov commented on 2024-02-10T01:55:55Z
----------------------------------------------------------------

Line #1.    import gradio as gr

Gradio is not installed


eaidova commented on 2024-02-12T10:08:08Z
----------------------------------------------------------------

added to requirements

@eaidova Review Notebook App
Copy link
Collaborator Author

eaidova commented Feb 12, 2024

added to install requirements

View entire conversation on ReviewNB

@eaidova Review Notebook App
Copy link
Collaborator Author

eaidova commented Feb 12, 2024

added to requirements

View entire conversation on ReviewNB

@eaidova
Copy link
Collaborator Author

eaidova commented Feb 12, 2024

View / edit / reply to this conversation on ReviewNB

aleksandr-mokrov commented on 2024-02-10T01:55:54Z ----------------------------------------------------------------

Here is no a table of contents

added

eaidova
eaidova commented Feb 12, 2024
View reviewed changes
eaidova
eaidova commented Feb 12, 2024
View reviewed changes
@eaidova eaidova merged commit 4d251af into openvinotoolkit:main Feb 13, 2024
@eaidova eaidova deleted the ea/iinstantid branch February 13, 2024 05:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@apaniukov apaniukov Awaiting requested review from apaniukov apaniukov was automatically assigned from openvinotoolkit/openvino_notebooks-developers

@itrushkin itrushkin Awaiting requested review from itrushkin itrushkin was automatically assigned from openvinotoolkit/openvino_notebooks-developers

@aleksandr-mokrov aleksandr-mokrov Awaiting requested review from aleksandr-mokrov aleksandr-mokrov was automatically assigned from openvinotoolkit/openvino_notebooks-developers

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@eaidova