flutter-tflite icon indicating copy to clipboard operation
flutter-tflite copied to clipboard

fix: final class for tflite bindings

Open mdejeans opened this issue 1 year ago • 5 comments

mdejeans avatar Aug 14 '24 07:08 mdejeans

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.

google-cla[bot] avatar Aug 14 '24 07:08 google-cla[bot]

So that's actually a generated class :) When we run the ffi script (https://github.com/tensorflow/flutter-tflite/blob/main/ffigen_tensorflow_lite.yaml) this gets overridden.

PaulTR avatar Aug 15 '24 18:08 PaulTR

@PaulTR, I wonder if there is an estimated timeline for when the tflite_flutter package will be updated to address the recent issues with the Dart upgrade?

s12dar avatar Aug 19 '24 13:08 s12dar

This should be a matter updating all the dependencies everywhere. What's happening is that ffigen is generating newer code than other aspects of the pubspec.yaml file are cool with.

craiglabenz avatar Aug 19 '24 18:08 craiglabenz

We can update the ffigen package to 8.0.0 and resolve this issue. This issue has been resolved after 8.0.0 version. https://pub.dev/packages/ffigen/changelog

chathurach avatar Aug 25 '24 15:08 chathurach

@PaulTR @chathurach gotcha!

mdejeans avatar Sep 02 '24 17:09 mdejeans