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

Enable export for visual prompting models #2955

Merged
merged 52 commits into from
Mar 6, 2024

Conversation

sungchul2
Copy link
Contributor

@sungchul2 sungchul2 commented Feb 21, 2024

Summary

This PR includes:

  • enable otx export
    • finetuning
    • zero-shot
  • enable IR inference
    • finetuning
    • zero-shot
  • enable standalone infer (not given reference feats, automatically execute learn first)
    • torch
    • OVIR

How to test

Checklist

  • I have added unit tests to cover my changes.​
  • I have added integration tests to cover my changes.​
  • I have added e2e tests for validation.
  • I have added the description of my changes into CHANGELOG in my target branch (e.g., CHANGELOG in develop).​
  • I have updated the documentation in my target branch accordingly (e.g., documentation in develop).
  • I have linked related issues.

License

  • I submit my code changes under the same Apache License that covers the project.
    Feel free to contact the maintainers if that's a concern.
  • I have updated the license header for each file (see an example below).
# Copyright (C) 2023 Intel Corporation
# SPDX-License-Identifier: Apache-2.0

@github-actions github-actions bot added TEST Any changes in tests OTX 2.0 labels Feb 21, 2024
@sungchul2 sungchul2 marked this pull request as ready for review February 29, 2024 04:47
@sungchul2 sungchul2 requested a review from sovrasov as a code owner March 4, 2024 16:09
@sungchul2 sungchul2 force-pushed the v2_add_vpm_export branch from e521fd7 to 05bfdfa Compare March 5, 2024 05:13
Copy link
Contributor

@eunwoosh eunwoosh left a comment

Choose a reason for hiding this comment

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

Thanks for your work :) I left some comments. Please take a look

src/otx/core/exporter/native.py Outdated Show resolved Hide resolved
src/otx/core/model/entity/base.py Outdated Show resolved Hide resolved
src/otx/core/model/entity/base.py Outdated Show resolved Hide resolved
src/otx/core/model/module/anomaly/anomaly_lightning.py Outdated Show resolved Hide resolved
src/otx/core/model/module/anomaly/anomaly_lightning.py Outdated Show resolved Hide resolved
src/otx/core/model/module/visual_prompting.py Outdated Show resolved Hide resolved
src/otx/core/model/module/visual_prompting.py Outdated Show resolved Hide resolved
src/otx/core/model/entity/visual_prompting.py Outdated Show resolved Hide resolved
@sungchul2 sungchul2 force-pushed the v2_add_vpm_export branch from 7c69585 to 2d75684 Compare March 5, 2024 14:15
@sungchul2 sungchul2 force-pushed the v2_add_vpm_export branch from 7cbd910 to a80aae2 Compare March 5, 2024 14:38
@sungchul2 sungchul2 requested a review from eunwoosh March 6, 2024 00:21
@sungchul2 sungchul2 merged commit fda118e into openvinotoolkit:v2 Mar 6, 2024
10 checks passed
@sungchul2 sungchul2 deleted the v2_add_vpm_export branch March 6, 2024 01:41
@sungchul2 sungchul2 restored the v2_add_vpm_export branch March 6, 2024 02:10
@sungchul2 sungchul2 deleted the v2_add_vpm_export branch July 3, 2024 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TEST Any changes in tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants