Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Replace use of asExternalTypedData with asTypedList. #31

Merged
merged 2 commits into from
Oct 15, 2019

Conversation

sjindel-google
Copy link
Contributor

No description provided.

Copy link

@mkustermann mkustermann left a comment

Choose a reason for hiding this comment

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

Consider adding a changelog entry

@sjindel-google sjindel-google merged commit ff18d20 into master Oct 15, 2019
@@ -1,3 +1,11 @@
# 0.3.1-dev.2

* Replaced uses of deprecated `asExternalTypedData` with `asTypedList`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this change the min. Dart SDK requirements for this package? If so, we should update the pubspec as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't think asTypedList has been published in a dev release yet, but we needed to make the change to ship D26, since this package is used in the SDK and we want to remove deprecated methods before shipping.

After shipping we should update the minimum constraint to D26 /cc @dcharkes @mkustermann

truongsinh added a commit to truongsinh/tflite_native that referenced this pull request Nov 13, 2019
* master:
  Update landed PR with new API (dart-archive#32)
  Resize tensor (dart-archive#24)
  Replace use of asExternalTypedData with asTypedList. (dart-archive#31)
  Remove type argument from structs (dart-archive#30)
  Remove use of deprecated FFI features via package:ffi. (dart-archive#29)
  Enable ci testing on osx (dart-archive#27)
  Whitespace cleanup in .travis.yml and README.md
  Enable and fix a number of lints
  significantly improve performance by `asExternalTypedData` (dart-archive#23)
  v0.3.0
  Issue 18 - Update to latest tflite C API (dart-archive#21)
  Update README.md
  Use Dart 2.5.0 as oldest required pub version (dart-archive#20)

# Conflicts:
#	lib/src/bindings/model.dart
#	lib/src/interpreter.dart
#	lib/src/model.dart
#	lib/src/tensor.dart
#	test/tflite_test.dart
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants