Skip to content

Conversation

@mbencer
Copy link
Contributor

@mbencer mbencer commented Feb 21, 2025

This commit introduces a new tool for Circle model resizing.

ONE-DCO-1.0-Signed-off-by: Mateusz Bencer [email protected]

Issue: #14791

This commit introduces a new tool for Circle model resizing.

ONE-DCO-1.0-Signed-off-by: Mateusz Bencer <[email protected]>
@mbencer mbencer added the PR/NO MERGE Please don't merge. I'm still working on this :) label Feb 21, 2025
@mbencer mbencer removed the PR/NO MERGE Please don't merge. I'm still working on this :) label Feb 27, 2025
@mbencer mbencer requested a review from a team February 27, 2025 08:17
@mbencer mbencer requested a review from jinevening April 23, 2025 11:18
namespace
{

void change_single_input_shape(luci::CircleInput *circle_input, const Shape &new_shape)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jinevening One thing to confirm here. Should circle-resizer also modify shape_signature attribute? I think no because it's a something specific for TF but I would like to confirm.

Copy link
Collaborator

Choose a reason for hiding this comment

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

circle uses shape_signature as well. PTAL luci/import/src/Importer.cpp.

(Not 100% sure) But you may not have to do further modification, because shape signature may be automatically set by encodeShapeSignature in CircleTensorExporter.cpp.

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.

2 participants