Skip to content

Conversation

@arkq
Copy link
Contributor

@arkq arkq commented Oct 22, 2025

This commits fixes the tensorinfo binding, so it will be possible to set the data type value in the Python code. The tensorinfo refactoring #16199 might take some time, so at least make current implementation functional.

Previous behavior:

>>> import onert
>>> ti = onert.tensorinfo()
>>> ti.dtype = 'int32'
>>> ti.dtype
''

Behavior after the fix:

>>> import onert
>>> ti = onert.tensorinfo()
>>> ti.dtype = 'int32'
>>> ti.dtype
'int32'

ONE-DCO-1.0-Signed-off-by: Arkadiusz Bokowy a.bokowy@samsung.com

This commits fixes the tensorinfo binding, so it will be possible to
set the data type value in the Python code.

ONE-DCO-1.0-Signed-off-by: Arkadiusz Bokowy <a.bokowy@samsung.com>
Copy link
Contributor

@hseok-oh hseok-oh left a comment

Choose a reason for hiding this comment

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

LGTM

@hseok-oh hseok-oh requested a review from ragmani October 23, 2025 06:20
Copy link
Contributor

@ragmani ragmani left a comment

Choose a reason for hiding this comment

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

LGTM

@hseok-oh hseok-oh merged commit 7989bef into Samsung:master Oct 24, 2025
10 checks passed
@arkq arkq deleted the tinfo-fix-dtype branch October 24, 2025 07:34
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