Skip to content

Conversation

@xin3he
Copy link
Contributor

@xin3he xin3he commented Jan 23, 2024

Type of Change

enhancement

Description

refactor folder of torch 3x API and add precision check UTs for RTN.

Expected Behavior & Potential Risk

UT pass

@xin3he xin3he changed the base branch from xinhe/patch-1 to master January 23, 2024 02:04
@xin3he xin3he marked this pull request as draft January 23, 2024 02:19
@xin3he xin3he marked this pull request as draft January 23, 2024 02:19
@xin3he xin3he force-pushed the xinhe/patch-2 branch 2 times, most recently from d90a953 to 335295f Compare January 25, 2024 06:58
@xin3he xin3he marked this pull request as ready for review January 25, 2024 07:00
@xin3he
Copy link
Contributor Author

xin3he commented Jan 25, 2024

This PR is based on #1557 and part of #1549

@xin3he xin3he force-pushed the xinhe/patch-2 branch 2 times, most recently from 3c7ea1b to ce10e9c Compare January 25, 2024 08:16
@xin3he xin3he changed the title refactor folder of torch 3x API refactor folder of torch 3x API and refine RTN Jan 25, 2024
Copy link
Contributor

@yiliu30 yiliu30 left a comment

Choose a reason for hiding this comment

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

Leave some comments, others are LGTM.

Signed-off-by: xin3he <[email protected]>
Signed-off-by: xin3he <[email protected]>
@yiliu30 yiliu30 self-requested a review January 26, 2024 08:00
@Kaihui-intel Kaihui-intel self-requested a review January 26, 2024 08:01
Copy link
Contributor

@yiliu30 yiliu30 left a comment

Choose a reason for hiding this comment

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

The overall folder structure follows the layout outlined in #1445 (comment). I've added comments specific to Torch.

I assume that the EXT test and UT coverage guarantee correctness and did not delve into the implementation details.

Signed-off-by: chensuyue <[email protected]>
chensuyue and others added 3 commits January 29, 2024 21:23
Signed-off-by: chensuyue <[email protected]>
Signed-off-by: xin3he <[email protected]>
@chensuyue chensuyue merged commit 7bf89eb into master Jan 30, 2024
@chensuyue chensuyue deleted the xinhe/patch-2 branch January 30, 2024 07:04
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.

4 participants