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.

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

micro: port op SPACE_TO_DEPTH from lite #45824

Closed
rkuester opened this issue Dec 18, 2020 · 2 comments
Closed

micro: port op SPACE_TO_DEPTH from lite #45824

rkuester opened this issue Dec 18, 2020 · 2 comments
Assignees
Labels
comp:micro Related to TensorFlow Lite Microcontrollers type:bug Bug

Comments

@rkuester
Copy link
Contributor

@tensorflow/micro

This issue tracks my work porting operator SPACE_TO_DEPTH from lite to micro. The port will be submitted in a number of PRs. Here's a rough flight plan in the style of #45306:

  • PR 1: Extract the code for parsing the op from a flatbuffer out of ParseOpDataTfLite in tensorflow/lite/core/api/flatbuffer_conversions.cc into a standalone function that can be called from micro's op resolver
  • PR 2: Extract the reference implementation out of tensorflow/lite/kernels/internal/reference/reference_ops.h into its own header which can be included without dragging in reference_ops.h's dependences
  • PR 3: Copy operator from lite to micro without making any changes or including in the build
  • PR 4: Delete extra code from the micro copy of the operator
  • PR 5: Port micro copy of operator as necessary and add a corresponding test
@rkuester rkuester added the comp:micro Related to TensorFlow Lite Microcontrollers label Dec 18, 2020
@rkuester rkuester assigned rkuester and unassigned Saduf2019 Dec 18, 2020
@rkuester rkuester changed the title micro: port op SPACE_TO_DEPTH from lite to micro micro: port op SPACE_TO_DEPTH from lite Dec 18, 2020
@rkuester
Copy link
Contributor Author

rkuester commented Jun 2, 2021

This issue has been migrated to the tflite-micro repository as tensorflow/tflite-micro#113.

@rkuester rkuester closed this as completed Jun 2, 2021
@google-ml-butler
Copy link

Are you satisfied with the resolution of your issue?
Yes
No

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:micro Related to TensorFlow Lite Microcontrollers type:bug Bug
Projects
None yet
Development

No branches or pull requests

3 participants