-
Notifications
You must be signed in to change notification settings - Fork 90
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 OCOS_ENABLE_VENDOR_IMAGE_CODECS compile definition, even if VISION is disabled #849
Conversation
@microsoft-github-policy-service agree company="Microsoft" |
/azp run "onnxruntime-extensions.CI" |
No pipelines are associated with this pull request. |
/azp run onnxruntime-extensions.CI |
Azure Pipelines successfully started running 1 pipeline(s). |
_DEFAULT_CODEC_ENABLE was added for these platforms which doesn't have native APIs for image codecs so that we don't need explicitly turn this flag ON or OFF for pipelines on different platforms. If this PR can resolve issue. we can merge this PR to unblock you since this flag was OFF now so it might not break any pipeline. Moreover, OCOS_ENABLE_VENDOR_IMAGE_CODECS cannot be ON by default is due to missing encoder support, with #848, we will turn on this flag. |
ok. so #848 will turn on OCOS_ENABLE_VENDOR_IMAGE_CODECS by default. I can wait for that PR |
libpng
andlibjpeg
on Windows and Apple, ifOCOS_ENABLE_VENDOR_IMAGE_CODECS
is enabledOCOS_ENABLE_VENDOR_IMAGE_CODECS
compile definition whenOCOS_ENABLE_VISION
is set, assuming that decoders are not used ifVISION
is not enabled. That's not the case becauseshared\api\image_processor.cc
is added forOCOS_ENABLE_C_API
(CMakeLists.txt:761
) andOCOS_BUILD_SHARED_LIB=OFF
(CMakeLists.txt:873
)In this change:
-DOCOS_ENABLE_VENDOR_IMAGE_CODECS
is performed even ifVISION
is not enabled._DEFAULT_CODEC_ENABLE
, andif(_DEFAULT_CODEC_ENABLE)
as it is redundant given it's preceded byset(_DEFAULT_CODEC_ENABLE ON)