Skip to content

Commit 9971d35

Browse files
metascroyfacebook-github-bot
authored andcommitted
Re-introduce some logging after backout (#14526)
Summary: Since D80715432 landed, we also landed D81256207 to add more logging. To prevent conflicts, we backout in 3 steps: * In diff 1/3, we backout D81256207, which added more logging * In diff 2/3, we backout D80715432, which is the problematic diff we're trying to revert * In diff 3/3, we introduce some of the logging we backed out in diff 1/3 Reviewed By: mergennachin Differential Revision: D82581441
1 parent 9283b4e commit 9971d35

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

backends/apple/coreml/runtime/delegate/ETCoreMLModelManager.mm

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -436,11 +436,13 @@ - (nullable NSURL *)compiledModelURLWithIdentifier:(NSString *)identifier
436436
switch (modelAssetType.value()) {
437437
case ModelAssetType::CompiledModel: {
438438
// Model is already compiled.
439+
ETCoreMLLogInfo("The model in the pte file is pre-compiled. Skipping compilation.");
439440
return modelURL;
440441
}
441442

442443
case ModelAssetType::Model: {
443444
// Compile the model.
445+
ETCoreMLLogInfo("The model in the pte file is not pre-compiled. Compiling with a 5 min timeout.");
444446
NSURL *compiledModelURL = [ETCoreMLModelCompiler compileModelAtURL:modelURL
445447
maxWaitTimeInSeconds:(5 * 60)
446448
error:error];

0 commit comments

Comments
 (0)