Skip to content

Build C-APIs for published native DLL#1017

Merged
hanbitmyths merged 4 commits intomicrosoft:mainfrom
amirkutcher:akutcher/update_build
Dec 10, 2025
Merged

Build C-APIs for published native DLL#1017
hanbitmyths merged 4 commits intomicrosoft:mainfrom
amirkutcher:akutcher/update_build

Conversation

@amirkutcher
Copy link
Contributor

This change enables building the C-APIs within the published ortextension.dll, so tokenization functionalities could be used for modules that does not hold tokenizer nodes, and required external tokenization prior to inference.

@amirkutcher amirkutcher requested a review from a team as a code owner December 10, 2025 11:05
@amirkutcher
Copy link
Contributor Author

/azp run onnxruntime-extensions.CI

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 1017 in repo microsoft/onnxruntime-extensions

@amirkutcher
Copy link
Contributor Author

/azp run onnxruntime-extensions.CI

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 1017 in repo microsoft/onnxruntime-extensions

@apsonawane
Copy link
Contributor

Pipelines are failing because of this error:

"C:\Program Files\CMake\bin\cmake.exe" -G "Visual Studio 17 2022"  -DOCOS_ENABLE_CTEST=ON -DOCOS_ONNXRUNTIME_VERSION="1.18.1" -DONNXRUNTIME_PKG_DIR=.\onnxruntime-win-x64-1.18.1 -DOCOS_ENABLE_C_API -B out\Windows -S .
CMake Error: Parse error in command line argument: OCOS_ENABLE_C_API
 Should be: VAR:type=value

@hanbitmyths hanbitmyths merged commit abae974 into microsoft:main Dec 10, 2025
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants