Skip to content

Conversation

@stamalakhov
Copy link
Contributor

This commit introduces MPQSelectionPanel for manual mixed precision quantization task.

Its correctness is tested in #1530

Fresh draft: #1530
Full draft: #1505
Related: #1491

ONE-vscode-DCO-1.0-Signed-off-by: s.malakhov [email protected]

@stamalakhov stamalakhov self-assigned this Apr 19, 2023
This commit introduces MPQSelectionPanel for manual mixed precision quantization task.

ONE-vscode-DCO-1.0-Signed-off-by: s.malakhov <[email protected]>
@stamalakhov stamalakhov force-pushed the manual_MPQ_graph_selector_br branch from 43be3bb to 5701061 Compare April 19, 2023 14:33
Copy link
Contributor

@seanshpark seanshpark left a comment

Choose a reason for hiding this comment

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

LGTM

extends CircleGraphCtrl
implements CircleGraphEvent
{
public static readonly viewType = "one.viewer.mpq";
Copy link
Contributor

Choose a reason for hiding this comment

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

I wonder if it's 'viewer' or' 'editor'..?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@dayo09
For now its viewer with 'selector' abilities. I mean it doesn't have a document to edit. IMHO.

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh, Thanks for the explanation!

@seanshpark seanshpark merged commit 7d4fe5b into Samsung:main Apr 21, 2023
@stamalakhov stamalakhov deleted the manual_MPQ_graph_selector_br branch April 21, 2023 05: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