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

Fixed compilation for Win32 #3803

Merged
merged 1 commit into from
Jan 12, 2021
Merged

Fixed compilation for Win32 #3803

merged 1 commit into from
Jan 12, 2021

Conversation

ilya-lavrenov
Copy link
Contributor

No description provided.

@ilya-lavrenov ilya-lavrenov added category: build OpenVINO cmake script / infra platform: win32 OpenVINO on Windows platform labels Jan 11, 2021
@ilya-lavrenov ilya-lavrenov added this to the 2021.3 milestone Jan 11, 2021
@ilya-lavrenov ilya-lavrenov mentioned this pull request Jan 11, 2021
Merged
@ilya-lavrenov ilya-lavrenov marked this pull request as ready for review January 11, 2021 15:41
@SDxKeeper
Copy link
Contributor

Looks like this is not enough to fix compilation issue on windows. Tests also use FusedOp
C:\a\1\openvino\inference-engine\tests\functional\inference_engine\cnn_network\convert_ngraph_to_cnn_network_tests.cpp(394): error C4996: 'ngraph::op::util::FusedOp::FusedOp': FusedOp approach was deprecated! Please use inheritance from usual Op instead of FusedOp

@ilya-lavrenov ilya-lavrenov requested a review from a team January 12, 2021 09:24
@ilya-lavrenov ilya-lavrenov force-pushed the win32-compilation branch 2 times, most recently from 8990b28 to 99ae1cc Compare January 12, 2021 10:46
@ilya-lavrenov ilya-lavrenov self-assigned this Jan 12, 2021
@ilya-lavrenov ilya-lavrenov merged commit 4d4efdc into openvinotoolkit:master Jan 12, 2021
@ilya-lavrenov ilya-lavrenov deleted the win32-compilation branch January 12, 2021 15:40
mryzhov pushed a commit to mryzhov/openvino that referenced this pull request Jan 14, 2021
@ilya-lavrenov @mryzhov
Fixed compilation for Win32 (openvinotoolkit#3803)
01d53c3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GlebKazantaev GlebKazantaev GlebKazantaev approved these changes

@ilyachur ilyachur ilyachur approved these changes

@vinograd47 vinograd47 vinograd47 approved these changes

Assignees

@ilya-lavrenov ilya-lavrenov

Labels
category: build OpenVINO cmake script / infra platform: win32 OpenVINO on Windows platform
Projects
None yet
Milestone
2021.3
Development

Successfully merging this pull request may close these issues.
5 participants
@ilya-lavrenov @SDxKeeper @vinograd47 @ilyachur @GlebKazantaev