Skip to content

add more recipes for AITK#16

Merged
xieofxie merged 13 commits into
mainfrom
hualxie/add_more
Aug 5, 2025
Merged

add more recipes for AITK#16
xieofxie merged 13 commits into
mainfrom
hualxie/add_more

Conversation

@xieofxie
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings July 30, 2025 04:03
@xieofxie xieofxie requested a review from a team as a code owner July 30, 2025 04:03

This comment was marked as outdated.

@xieofxie xieofxie requested a review from Copilot July 30, 2025 07:32
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds LICENSE files to multiple model directories and adds several new AITK model recipes, including Microsoft Phi-4 variants, DeepSeek R1 Distill models, and updates to existing configurations. The changes include both new model integrations and improvements to the JSON configuration structure by removing empty arrays.

  • Adds LICENSE files for compliance across multiple model directories
  • Introduces new AITK recipes for Phi-4 reasoning models and DeepSeek variants
  • Improves JSON configuration consistency by replacing empty arrays with proper formatting

Reviewed Changes

Copilot reviewed 186 out of 188 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
Multiple LICENSE files Adds appropriate license files (MIT, Apache 2.0, etc.) for model compliance
microsoft-Phi-4-* directories New AITK recipes for Phi-4 reasoning, reasoning-plus, and mini-instruct models
deepseek-ai-DeepSeek-R1-* directories New AITK recipes for DeepSeek R1 Distill models in 7B and 14B variants
Various _copy.json.config files JSON formatting improvements removing empty replacement arrays
Requirements.txt files Standardized requirement file headers and dependencies
Comments suppressed due to low confidence (1)

microsoft-Phi-4-mini-instruct/aitk/inference_model.json:2

  • The model name should match the actual model being configured. This should be 'Phi-4-mini-instruct-onnx' instead of 'Phi-3.5-mini-instruct-onnx'.
  "Name": "Phi-3.5-mini-instruct-onnx",

Comment thread microsoft-Phi-4-mini-instruct/aitk/phi4_ov_config.json Outdated
@xieofxie xieofxie requested a review from a team as a code owner August 4, 2025 02:25
fangyangci
fangyangci previously approved these changes Aug 4, 2025
@xieofxie xieofxie changed the title add LICENSEs, add more recipes for AITK add more recipes for AITK Aug 5, 2025
@xieofxie xieofxie merged commit 05d418a into main Aug 5, 2025
1 check passed
@xieofxie xieofxie deleted the hualxie/add_more branch August 5, 2025 02:20
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