Skip to content

Conversation

@veblush
Copy link
Collaborator

@veblush veblush commented Jan 12, 2026

BUG=n/a

@veblush veblush requested a review from a team as a code owner January 12, 2026 22:51
@veblush veblush force-pushed the flatbuffer-25.9.23 branch from fe00223 to 00f6b50 Compare January 13, 2026 06:09

TF_LITE_MICRO_TEST(IfShouldInvokeSubgraphConditionFalse) {
constexpr int kArenaSize = 5000;
constexpr int kArenaSize = 16384;
Copy link
Member

Choose a reason for hiding this comment

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

@veblush Thats a big increase in arena size. Any ideas why this expansion happened?


TF_LITE_MICRO_TEST(IfShouldNotOverwriteTensorAcrossSubgraphs) {
constexpr int kArenaSize = 5000;
constexpr int kArenaSize = 16384;
Copy link
Member

Choose a reason for hiding this comment

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

@veblush Same as line 138 question

+#endif
}
case FBT_VECTOR: return static_cast<double>(AsVector().size());
case FBT_BOOL:
Copy link
Member

Choose a reason for hiding this comment

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

FBT_BOOL is no more?


TF_LITE_MICRO_TEST(TestMultiSubgraphWithIf) {
constexpr int kArenaSize = 1024;
constexpr int kArenaSize = 2048;
Copy link
Member

Choose a reason for hiding this comment

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

@veblush Did the updated flatbuffers library cause the model schema to expand in size?

flatbuffers::FlatBufferBuilder* builder = BuilderInstance();
constexpr size_t tensor_shape_size = 1;
const int32_t tensor_shape[tensor_shape_size] = {size};
std::vector<int32_t> shape = {size};
Copy link
Member

Choose a reason for hiding this comment

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

@veblush will the verboten std::vector port to non x86 platforms?

@veblush veblush marked this pull request as draft January 13, 2026 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants