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

Load TfLite models correctly for s390x #2200

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yasiribmcon
Copy link

Proposed changes are an extension for tensorflow/tensorflow#60190.
These changes enable string tensors in TFLite models which were generated on LE machines to be loaded successfully on BE platforms.
Below test cases from TF serving test suite are getting rectified after applying this PR on s390x(BE machines):

//tensorflow_serving/servables/tensorflow:saved_model_bundle_factory_test
//tensorflow_serving/servables/tensorflow:tflite_interpreter_pool_test
//tensorflow_serving/servables/tensorflow:tflite_session_test

These changes will not cause any regression on LE/BE platforms.

Loading TfLite model from file(little endian) correctly for s390x(big endian) systems.

Signed-off-by: Yasir Ashfaq <[email protected]>
@yasiribmcon
Copy link
Author

Hi @gbaned

Could you please take a look at this PR when you have some time?

Thank you very much!

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.

1 participant