Skip to content
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

[EdgeTPU] Implement EdgeTPUCfgEditorPanel #1728

Closed
wants to merge 3 commits into from

Conversation

dayo09
Copy link
Contributor

@dayo09 dayo09 commented Nov 1, 2023

This PR includes three PRs for EdgeTPUCfgEditorPanel.

NOTE They will be merged separately (not squash-merge)


Waiting for #1725 to be merged
From #1726

hohee-hee and others added 3 commits November 1, 2023 21:08
This commit creates EdgeTPUCfgEditorPanel.ts that is a panel for .edgetpucfg.
- Create EdgetpuCfgEditorPanel.ts : Cfg Editor Panel for edgetpu cfg editor
- Add register in extension.ts
- Map a new view type with a new file extension - .edgetpucfg
- Error fix : Duplication of setDefaultValues causes error so changed the command name to "setDefaultEdgetpuValues".
- Add objects about setDefaultEdgetpuValues in Commands and Keybindings in package.json

ONE-vscode-DCO-1.0-Signed-off-by: hohee-hee <[email protected]>

Co-authored-by: Bumsoo Ko <[email protected]>
Co-authored-by: Hyeon-Uk <[email protected]>
For modularization of EdgeTPU, we decided to add a new file name extension for config file for edgetpu compiler.
Its name is 'edgetpucfg'.
'edgetpucfg' has different steps and options than the original cfg file.
Therefore, this commit adds a new directory and files for .edgetpucfg in /media/EdgetpuCfgEditor.

ONE-vscode-DCO-1.0-Signed-off-by: hohee-hee <[email protected]>

Co-authored-by: Bumsoo Ko <[email protected]>
Co-authored-by: 김현욱 <[email protected]>
ONE-vscode-DCO-1.0-Signed-off-by: Dayoung Lee <[email protected]>
@dayo09 dayo09 added the NO MERGE label Nov 1, 2023
@dayo09 dayo09 closed this Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants