Skip to content

[DOC] update coreml new opretors and flags#22958

Merged
wejoncy merged 20 commits intogh-pagesfrom
jicwen/pages
Dec 24, 2024
Merged

[DOC] update coreml new opretors and flags#22958
wejoncy merged 20 commits intogh-pagesfrom
jicwen/pages

Conversation

@wejoncy
Copy link
Contributor

@wejoncy wejoncy commented Nov 27, 2024

Description

Motivation and Context

@wejoncy wejoncy marked this pull request as ready for review November 27, 2024 07:10
@wejoncy wejoncy requested a review from skottmckay November 27, 2024 08:12
@wejoncy wejoncy linked an issue Nov 29, 2024 that may be closed by this pull request
@wejoncy wejoncy changed the title update coreml new opretors and flags [DOC] update coreml new opretors and flags Nov 29, 2024
wejoncy and others added 3 commits December 16, 2024 10:01
Co-authored-by: Scott McKay <skottmckay@gmail.com>
Co-authored-by: Scott McKay <skottmckay@gmail.com>
Copy link
Contributor

@skottmckay skottmckay left a comment

Choose a reason for hiding this comment

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

:shipit:

@wejoncy wejoncy merged commit af2f2d5 into gh-pages Dec 24, 2024
4 checks passed
@wejoncy wejoncy deleted the jicwen/pages branch December 24, 2024 02:13
@jcelerier
Copy link
Contributor

jcelerier commented Apr 12, 2025

the doc here is wrong and does not compile:

provider_options["ModelFormat"] = std::to_string("MLProgram");
provider_options["MLComputeUnits"] = std::to_string("ALL");
provider_options["RequireStaticInputShapes"] = std::to_string("0");
provider_options["EnableOnSubgraphs"] = std::to_string("0");

std::to_string is to convert a number (int, float etc.) to a string. It should be std::string or just passing the array as it will be converted automatically to std::string:

provider_options["ModelFormat"] = "MLProgram";
provider_options["MLComputeUnits"] = "ALL";
provider_options["RequireStaticInputShapes"] = "0";
provider_options["EnableOnSubgraphs"] = "0";

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.

[Documentation] CoreML python support and how to switch to MLProgram

3 participants