Skip to content

Conversation

@junseokShim
Copy link

Changes to align with TFLM conventions / review feedback

  • Flattened the namespace usage to namespace tflite { ... } and kept file-local helpers in an anonymous namespace
  • Renamed the output shape check helper to reflect its actual behavior (validation, not resizing)
  • Updated comment wording to refer to Memory Planner (not “Offline Memory Planner”)
  • Updated the micro test to use tflite::Register_ONE_HOT() via the standard registration header (no forward declaration)

bug=fixes #3078

@junseokShim junseokShim requested a review from a team as a code owner December 17, 2025 15:01
@google-cla
Copy link

google-cla bot commented Dec 17, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@junseokShim junseokShim force-pushed the origin/feature/port_onehot_operator branch from c1f82e8 to 0523508 Compare December 17, 2025 15:06
@junseokShim
Copy link
Author

@ddavis-2015
Hi! I’m an external contributor so I can’t apply CI labels.
one_hot_test passes locally and CLA is resolved.
Could you please add ci:run (or ci:run_full if needed) and cla: yes so CI can run? Thanks!****

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