-
Notifications
You must be signed in to change notification settings - Fork 0
UPSTREAM PR #16609: mtmd : Fix/Add non-ASCII file path support on Windows #124
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
base: main
Are you sure you want to change the base?
UPSTREAM PR #16609: mtmd : Fix/Add non-ASCII file path support on Windows #124
Conversation
|
Access the complete analysis in the LOCI Dashboard Performance Analysis SummaryOverviewAnalysis of LLaMA.cpp project comparing versions f60c3df0 and 96963f27 reveals minimal performance impact from recent changes. The modifications primarily address Windows file path handling for non-ASCII characters in the mtmd (multimodal) feature, with no changes to core inference functions. Key FindingsPerformance Metrics:
Core Function Impact: Power Consumption Analysis:
Technical Analysis: Code Review Insights: Inference Performance Impact: The analysis indicates stable performance for primary LLaMA.cpp functionality with localized, non-critical changes in auxiliary components. |
3 similar comments
|
Access the complete analysis in the LOCI Dashboard Performance Analysis SummaryOverviewAnalysis of LLaMA.cpp project comparing versions f60c3df0 and 96963f27 reveals minimal performance impact from recent changes. The modifications primarily address Windows file path handling for non-ASCII characters in the mtmd (multimodal) feature, with no changes to core inference functions. Key FindingsPerformance Metrics:
Core Function Impact: Power Consumption Analysis:
Technical Analysis: Code Review Insights: Inference Performance Impact: The analysis indicates stable performance for primary LLaMA.cpp functionality with localized, non-critical changes in auxiliary components. |
|
Access the complete analysis in the LOCI Dashboard Performance Analysis SummaryOverviewAnalysis of LLaMA.cpp project comparing versions f60c3df0 and 96963f27 reveals minimal performance impact from recent changes. The modifications primarily address Windows file path handling for non-ASCII characters in the mtmd (multimodal) feature, with no changes to core inference functions. Key FindingsPerformance Metrics:
Core Function Impact: Power Consumption Analysis:
Technical Analysis: Code Review Insights: Inference Performance Impact: The analysis indicates stable performance for primary LLaMA.cpp functionality with localized, non-critical changes in auxiliary components. |
|
Access the complete analysis in the LOCI Dashboard Performance Analysis SummaryOverviewAnalysis of LLaMA.cpp project comparing versions f60c3df0 and 96963f27 reveals minimal performance impact from recent changes. The modifications primarily address Windows file path handling for non-ASCII characters in the mtmd (multimodal) feature, with no changes to core inference functions. Key FindingsPerformance Metrics:
Core Function Impact: Power Consumption Analysis:
Technical Analysis: Code Review Insights: Inference Performance Impact: The analysis indicates stable performance for primary LLaMA.cpp functionality with localized, non-critical changes in auxiliary components. |
aa2fc28 to
0ad40ce
Compare
7d44551 to
8eaea62
Compare
Mirrored from ggml-org/llama.cpp#16609
Summary
Fixed path handling in the mtmd feature on Windows to correctly convert and process non-ASCII file paths.
Additionally, fixed a missing console initialization in mtmd-cli.
Issues Resolved
Failed to load .mmproj files with non-ASCII file paths
Failed to load image files with non-ASCII file paths
Incorrect path parsing in mtmd-cli when using
/image <filepath>with non-ASCII charactersComparison of behavior
Windows 11 Pro 24H2
b6756
This PR
Notes
The general argument-handling logic in llama.cpp also has issues with non-ASCII paths.
Since the impact area is broad, I will submit a separate PR to address this.