Skip to content

[NVIDIA] Update setup.py script for nvidia_plugin #923

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
Merged
andrei-kochin merged 2 commits into from
May 22, 2025
Merged

[NVIDIA] Update setup.py script for nvidia_plugin #923

andrei-kochin merged 2 commits into from
May 22, 2025

Conversation

@redradist
Copy link
Contributor

@redradist redradist commented Jun 7, 2024 •
edited
Loading

Updated version of the following PR #751

@redradist redradist requested a review from a team as a code owner June 7, 2024 08:06
@github-actions github-actions bot added category: NVIDIA plugin OpenVINO NVIDIA plugin dependencies Pull requests that update a dependency file labels Jun 7, 2024
@redradist redradist force-pushed the refactoring/update-setup-py-script branch 2 times, most recently from 031471f to 222d090 Compare June 7, 2024 08:11
@redradist redradist changed the title Update setup.py script for nvidia_plugin [NVIDIA] Update setup.py script for nvidia_plugin Jun 7, 2024
@ilya-lavrenov
Copy link
Contributor

ilya-lavrenov commented Jun 7, 2024

build_jenkins

@redradist redradist force-pushed the refactoring/update-setup-py-script branch from 222d090 to 9e1258a Compare June 7, 2024 08:45
@redradist
Copy link
Contributor Author

redradist commented Jun 10, 2024

build_jenkins

@ilya-lavrenov By some reason Jenkins job still not started

@ilya-lavrenov
Copy link
Contributor

ilya-lavrenov commented Jun 10, 2024

build_jenkins

@redradist
Copy link
Contributor Author

redradist commented Jun 10, 2024

@ilya-lavrenov Could you provide logs for ci/jenkins and ie_tests_gpu_nvidia_ubuntu20_release ?
by some reason it failed, but I have not touched cmake files

@ilya-lavrenov
Copy link
Contributor

ilya-lavrenov commented Jun 10, 2024

@ilya-lavrenov Could you provide logs for ci/jenkins and ie_tests_gpu_nvidia_ubuntu20_release ? by some reason it failed, but I have not touched cmake files

image

@redradist
Copy link
Contributor Author

redradist commented Jun 13, 2024 •
edited
Loading

@ilya-lavrenov Could you provide logs for ci/jenkins and ie_tests_gpu_nvidia_ubuntu20_release ? by some reason it failed, but I have not touched cmake files

image

@ilya-lavrenov @likholat As you can see in PR I do not touch tests.
Looks like they are broken already

It was already broken in the following PR #920

@redradist redradist force-pushed the refactoring/update-setup-py-script branch from d7e010f to 44dee44 Compare June 14, 2024 13:40
@redradist
Copy link
Contributor Author

redradist commented Jun 19, 2024 •
edited
Loading

@ilya-lavrenov Could you provide logs for ci/jenkins and ie_tests_gpu_nvidia_ubuntu20_release ? by some reason it failed, but I have not touched cmake files

image

@ilya-lavrenov As you can see in PR I do not touch tests. Looks like they are broken already

It was already broken in the following PR #920

@ilya-lavrenov Is there any updates ?

@redradist redradist requested a review from ilya-lavrenov June 27, 2024 09:12
@redradist
Copy link
Contributor Author

redradist commented Jul 9, 2024

@ilya-lavrenov Any updates ?

@p-durandin p-durandin enabled auto-merge (squash) July 10, 2024 05:53
auto-merge was automatically disabled January 11, 2025 14:33

Head branch was pushed to by a user without write access

@redradist redradist force-pushed the refactoring/update-setup-py-script branch 2 times, most recently from 58593e2 to ef1b18f Compare January 11, 2025 16:35
@redradist
Copy link
Contributor Author

redradist commented Jan 11, 2025

@ilya-lavrenov @p-durandin If PR is approved, could someone merge it once CI will be finished after rebase

@redradist
Copy link
Contributor Author

redradist commented May 21, 2025

@ilya-lavrenov @p-durandin If PR is approved, could someone merge it once CI will be finished after rebase

@ilya-lavrenov @p-durandin @andrei-kochin
...

@redradist redradist force-pushed the refactoring/update-setup-py-script branch from 5ae8240 to d7aeb50 Compare May 21, 2025 19:01
@andrei-kochin andrei-kochin merged commit a44463a into openvinotoolkit:master May 22, 2025
1 check passed
@andrei-kochin
Copy link

andrei-kochin commented May 22, 2025 •
edited
Loading

@redradist here you go! sorry for the delay
Better late than never (=

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@p-durandin p-durandin p-durandin approved these changes

@ilya-lavrenov ilya-lavrenov Awaiting requested review from ilya-lavrenov

Assignees

No one assigned

Labels

category: NVIDIA plugin OpenVINO NVIDIA plugin dependencies Pull requests that update a dependency file

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@redradist @ilya-lavrenov @andrei-kochin @p-durandin @lohika-denis-kotov