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.

Sign up for GitHub

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

Jump to bottom

[onert/backend] Add LayerScopeTensors into TensorRegistry #14024

Merged
merged 1 commit into from
Sep 24, 2024
Merged

[onert/backend] Add LayerScopeTensors into TensorRegistry #14024

merged 1 commit into from
Sep 24, 2024

Conversation

zetwhite
Copy link
Contributor

This PR adds LayerScopeTensors into TensorRegistry.

ONE-DCO-1.0-Signed-off-by: seunghui youn sseung.youn@samsung.com

draft : #13486
for : #13282

@zetwhite zetwhite added the approval: 2 Require at least 2 approvals label Sep 13, 2024
@zetwhite zetwhite requested review from ragmani and a team September 13, 2024 08:37
@zetwhite zetwhite added the PR/ready for review It is ready to review. Please review it. label Sep 13, 2024
@zetwhite zetwhite mentioned this pull request Sep 13, 2024
Open
jyoungyun
jyoungyun previously approved these changes Sep 19, 2024
View reviewed changes
Copy link
Contributor

@jyoungyun jyoungyun left a comment

Choose a reason for hiding this comment

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

LGTM

@zetwhite
[onert/backend] Add LayerScopeTensors into TensorRegistry
723bc1a 
This PR adds LayerScopeTensors into TensorRegistry.

ONE-DCO-1.0-Signed-off-by: seunghui youn <sseung.youn@samsung.com>

draft : Samsung#13486
for : Samsung#13282
@jyoungyun
Copy link
Contributor

@nnfw-bot test onert-cross-aarch64-release
@nnfw-bot test onert-cross-debug
@nnfw-bot test onert-cross-release
@nnfw-bot test tizen-gbs

1 similar comment
@zetwhite
Copy link
Contributor Author

@nnfw-bot test onert-cross-aarch64-release
@nnfw-bot test onert-cross-debug
@nnfw-bot test onert-cross-release
@nnfw-bot test tizen-gbs

@zetwhite
Copy link
Contributor Author

zetwhite commented Sep 23, 2024 •
edited
Loading 

[2024-09-23T02:20:47.274Z] ERROR: docker login failed

[2024-09-23T02:20:47.298Z] Setting status of 723bc1afc138fa55d88fa175cd6b1c44265bda77 to FAILURE with url http://13.125.34.93:8080/job/nnfw/job/master/job/pr-onert-cross-release/8846/display/redirect and message: 'Failed. Comment "@nnfw-bot test onert-cross-release" if you want to rerun

[2024-09-23T02:20:47.298Z]  '

[2024-09-23T02:20:47.298Z] Using context: onert-cross-release

[2024-09-23T02:20:47.787Z] Finished: FAILURE

@Samsung/one_onert Does anyone know what's happening here? It's been like this since last Friday. 😢

@hseok-oh
Copy link
Contributor

@nnfw-bot test onert-cross-aarch64-release onert-cross-debug onert-cross-release tizen-gbs

@zetwhite
Copy link
Contributor Author

PTAL @jyoungyun @ragmani
Thank you in advance :)

@hseok-oh hseok-oh merged commit 4d12964 into Samsung:master Sep 24, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ragmani ragmani ragmani approved these changes

@jyoungyun jyoungyun Awaiting requested review from jyoungyun

Assignees
No one assigned
Labels
approval: 2 Require at least 2 approvals PR/ready for review It is ready to review. Please review it.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@zetwhite @jyoungyun @hseok-oh @ragmani