-
Notifications
You must be signed in to change notification settings - Fork 141
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
[tools/circle_plus_gen] Add tparam inject feature #13027
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
632aaac
to
7ed4117
Compare
c101df0
to
2f41c88
Compare
2f41c88
to
5fb8578
Compare
This PR adds a feature - inject training parameters to a circle plus instance - export circle plus instance as a *.circle file ONE-DCO-1.0-Signed-off-by: seunghui youn <sseung.youn@samsung.com> * related issue : Samsung#12650
5fb8578
to
1e83e10
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Co-authored-by: Jiyoung Giuliana Yun <wldudyun10@gmail.com>
@@ -7,6 +8,7 @@ | |||
|
|||
class TrainParam(): | |||
'''Wrapper class of circle_traninfo_generated.ModelTrainingT''' | |||
TRAINING_PARAM_IDENTIFIER = b"CTR0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(note)
ONE/runtime/libs/circle-schema/include/circle_traininfo.fbs
Lines 18 to 21 in 14333cd
// file_identifier for circle training. | |
// The last character is a number which corresponds to the major version. | |
// This identifier number should be same if it does not break compatibility. | |
file_identifier "CTR0"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PR adds a feature - inject training parameters to a circle plus instance - export circle plus instance as a *.circle file ONE-DCO-1.0-Signed-off-by: seunghui youn <sseung.youn@samsung.com> Co-authored-by: Jiyoung Giuliana Yun <wldudyun10@gmail.com>
This PR adds a feature
ONE-DCO-1.0-Signed-off-by: seunghui youn sseung.youn@samsung.com