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

Updated Bert Qa Android to use Task Library and Interpreter #327

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

SunitRoy2703
Copy link
Contributor

@SunitRoy2703 SunitRoy2703 commented Jun 24, 2021

  1. Updated the bert qa android to use Task Library and Interpreter
  2. Wrote Instrumented tests for the App
  3. Updated the readme file
  4. Added gitignore to the directory
  5. Updated minSDK to 21 from 26, to support older devices
  6. Added Explore the code file

@google-ml-butler google-ml-butler bot added the size:XL CL Change Size: Extra Large label Jun 24, 2021
@google-cla google-cla bot added the cla: yes CLA has been signed label Jun 24, 2021
@SunitRoy2703 SunitRoy2703 changed the title Updated Bert Qa Android to use Task Library Updated Bert Qa Android to use Task Library and Interpreter Jul 23, 2021
that leverages the out-of-box API from the
[TensorFlow Lite Task Library](https://www.tensorflow.org/lite/inference_with_metadata/task_library/bert_question_answerer),
and
[`lib_interpreter`](https://github.com/SunitRoy2703/examples/tree/bertQa-android-task-lib/lite/examples/bert_qa/android/lib_interpreter)
Copy link
Member

Choose a reason for hiding this comment

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

Use relative paths to avoid dependencies on repo path. eg:

replace - https://github.com/SunitRoy2703/examples/tree/bertQa-android-task-lib/lite/examples/bert_qa/android/lib_interpreter with just
lib_interpreter as the current folder is ../examples/bert_qa/android

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you for reviewing!

I have replaced the links with relative links

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes CLA has been signed size:XL CL Change Size: Extra Large
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants