Merged
Conversation
copybara-service
bot
force-pushed
the
test_399373571
branch
from
be12d36 to
8ddda6b
Compare
marcvanzee
changed the title
copybara-service
bot
force-pushed
the
test_399373571
branch
from
8ddda6b to
dc9b36f
Compare
copybara-service
bot
changed the title
copybara-service
bot
force-pushed
the
test_399373571
branch
2 times, most recently
from
eab674f to
ce29692
Compare
Codecov Report
@@ Coverage Diff @@
## main #1597 +/- ##
=======================================
Coverage 82.75% 82.75%
=======================================
Files 68 68
Lines 5583 5583
=======================================
Hits 4620 4620
Misses 963 963 Continue to review full report at Codecov.
|
…oth TFLite and TFjs. The evaluation framework has the following features: * It is easy to add new Modules of examples since each Module is specified using a few lines of code (see `examples.py`). * It is easy to add new converters since each converter is represented as a function (see `converters.py`). For instance, we could add the MLIR-based converter that the TFLite team is currently working on. * The framework outputs a Markdown table (see `README.md`). The framework has the following limitations: * We only evaluate whether a Module converts, we do not compare any outputs between the converted model and the original model. This will require more effort, and it seems like we can do this as a follow-up if necessary (once a good fraction of ops are converted). * If an example is missing multiple ops, then only the first missing op is reported. We could improve this by implementing mocked versions of non-working ops, which only output the right shapes. We could also consider doing this as a follow-up. PiperOrigin-RevId: 399373571
copybara-service
bot
force-pushed
the
test_399373571
branch
from
ce29692 to
1820750
Compare
marcvanzee
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two small fixes in sst and vae