Skip to content

Conversation

@HeJunyan
Copy link

@HeJunyan HeJunyan commented Nov 17, 2025

The Deepseek OCR model uses fixed input tensor shape, so there is no need for padding.
Its input is different from the other MM models so a new fake data set is added to warm it up.

@HeJunyan HeJunyan force-pushed the improvement_performance_deepseek_ocr branch 2 times, most recently from 8c391a6 to b560b50 Compare December 8, 2025 14:15
@HeJunyan HeJunyan force-pushed the improvement_performance_deepseek_ocr branch from b560b50 to ebf2220 Compare December 8, 2025 14:26
'''

def __init__(self, is_batch_based):
def __init__(self, is_batch_based, sub_image_list=None):
Copy link

Choose a reason for hiding this comment

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

suggest to add the description of the argument of is_batch_based and sub_image_list in the defination.
sub_image_list seems like a list of image objects. But from line 117, maybe it is a list of integers?

Copy link
Author

Choose a reason for hiding this comment

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

Done

@HeJunyan HeJunyan force-pushed the improvement_performance_deepseek_ocr branch from ebf2220 to 51dfa56 Compare December 8, 2025 15:55
@HeJunyan
Copy link
Author

HeJunyan commented Dec 8, 2025

I verified the PR in full warmup mode, the result is unchanged. So it is OK.

@HeJunyan HeJunyan force-pushed the improvement_performance_deepseek_ocr branch 2 times, most recently from 44f67ad to e1ab76b Compare December 8, 2025 16:13
@HeJunyan HeJunyan force-pushed the improvement_performance_deepseek_ocr branch from e1ab76b to 363a85a Compare December 8, 2025 16:23
Copy link

@czhu15 czhu15 left a comment

Choose a reason for hiding this comment

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

LGTM

@czhu15 czhu15 merged commit d38f197 into HabanaAI:aice/v1.22.0 Dec 10, 2025
1 check 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.

2 participants