-
Notifications
You must be signed in to change notification settings - Fork 10
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
Bringup tt-torch models in forge #1314
Conversation
|
|
|
1 similar comment
|
66ff210
to
09430d9
Compare
|
|
|
|
899912e
to
823a81e
Compare
|
3 similar comments
|
|
|
823a81e
to
da83043
Compare
|
3 similar comments
|
|
|
e8ebdde
to
8b18209
Compare
|
1 similar comment
|
|
fb01ef7
to
5bd73b7
Compare
|
|
1 similar comment
|
|
LGTM |
verify(inputs, framework_model, compiled_model) | ||
|
||
# Inference | ||
output = compiled_model(*inputs) |
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.
Take a look at this PR. When this merges into main, verify()
would also return outputs of the forward pass.
|
||
|
||
def load_input(): | ||
url = "https://i.postimg.cc/ZKwLg2Gw/367-14.png" |
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.
I think it would be nice to have one centralized folder e.g. data_inputs
where we would store few images so all models would use the same inputs.
5bd73b7
to
42c0352
Compare
42c0352
to
eed2d0d
Compare
@vkovinicTT. I created a separate issue(#1364) for this suggestion. |
Awesome, thanks :) |
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, just waiting for the verify()
change.
|
|
|
|
|
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! Thanks for the PR @kamalrajkannan78
Yep, go ahead. |
Summary
This PR addresses Issue #1321 by porting models from tt-torch to tt-forge. The current compilation status of these models are available in the logs.
A list of skipped models along with the reasons for their exclusion can be found in Issue #1339. Any models that were missed in this PR will be included in PR #1337.
Note:
Logs: