keras-team / keras-core

A multi-backend implementation of the Keras API, with support for TensorFlow, JAX, and PyTorch.
Apache License 2.0
1.27k stars 115 forks source link

Add support for Tensorflow SparseTensors: merging layers. #925

Closed hertschuh closed 10 months ago

hertschuh commented 10 months ago

Added tf.SparseTensor support for ops:

Added tf.SparseTensor support for merging layers:

Note that the Dot merging layer will be addressed in a separate PR.

codecov[bot] commented 10 months ago

Codecov Report

Patch coverage: 100.00% and project coverage change: -11.51% :warning:

Comparison is base (0aa999b) 83.65% compared to head (4698c0e) 72.14%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #925 +/- ## =========================================== - Coverage 83.65% 72.14% -11.51% =========================================== Files 318 318 Lines 28666 28735 +69 Branches 5464 5483 +19 =========================================== - Hits 23980 20731 -3249 - Misses 3168 6633 +3465 + Partials 1518 1371 -147 ``` | [Flag](https://app.codecov.io/gh/keras-team/keras-core/pull/925/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team) | Coverage Δ | | |---|---|---| | [keras_core](https://app.codecov.io/gh/keras-team/keras-core/pull/925/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team) | `72.09% <100.00%> (-11.46%)` | :arrow_down: | | [keras_core-jax](https://app.codecov.io/gh/keras-team/keras-core/pull/925/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team) | `67.04% <48.10%> (-0.07%)` | :arrow_down: | | [keras_core-numpy](https://app.codecov.io/gh/keras-team/keras-core/pull/925/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team) | `?` | | | [keras_core-tensorflow](https://app.codecov.io/gh/keras-team/keras-core/pull/925/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team) | `66.86% <100.00%> (+0.07%)` | :arrow_up: | | [keras_core-torch](https://app.codecov.io/gh/keras-team/keras-core/pull/925/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team) | `?` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files Changed](https://app.codecov.io/gh/keras-team/keras-core/pull/925?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team) | Coverage Δ | | |---|---|---| | [keras\_core/backend/tensorflow/numpy.py](https://app.codecov.io/gh/keras-team/keras-core/pull/925?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team#diff-a2VyYXNfY29yZS9iYWNrZW5kL3RlbnNvcmZsb3cvbnVtcHkucHk=) | `94.60% <100.00%> (+0.68%)` | :arrow_up: | | [keras\_core/layers/merging/add.py](https://app.codecov.io/gh/keras-team/keras-core/pull/925?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team#diff-a2VyYXNfY29yZS9sYXllcnMvbWVyZ2luZy9hZGQucHk=) | `100.00% <100.00%> (ø)` | | | [keras\_core/layers/merging/average.py](https://app.codecov.io/gh/keras-team/keras-core/pull/925?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team#diff-a2VyYXNfY29yZS9sYXllcnMvbWVyZ2luZy9hdmVyYWdlLnB5) | `92.30% <100.00%> (+0.64%)` | :arrow_up: | | [keras\_core/layers/merging/base\_merge.py](https://app.codecov.io/gh/keras-team/keras-core/pull/925?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team#diff-a2VyYXNfY29yZS9sYXllcnMvbWVyZ2luZy9iYXNlX21lcmdlLnB5) | `41.08% <100.00%> (+2.37%)` | :arrow_up: | | [keras\_core/layers/merging/multiply.py](https://app.codecov.io/gh/keras-team/keras-core/pull/925?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team#diff-a2VyYXNfY29yZS9sYXllcnMvbWVyZ2luZy9tdWx0aXBseS5weQ==) | `92.30% <100.00%> (+0.64%)` | :arrow_up: | | [keras\_core/layers/merging/subtract.py](https://app.codecov.io/gh/keras-team/keras-core/pull/925?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team#diff-a2VyYXNfY29yZS9sYXllcnMvbWVyZ2luZy9zdWJ0cmFjdC5weQ==) | `81.25% <100.00%> (+1.25%)` | :arrow_up: | | [keras\_core/ops/numpy.py](https://app.codecov.io/gh/keras-team/keras-core/pull/925?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team#diff-a2VyYXNfY29yZS9vcHMvbnVtcHkucHk=) | `94.49% <100.00%> (+0.04%)` | :arrow_up: | ... and [48 files with indirect coverage changes](https://app.codecov.io/gh/keras-team/keras-core/pull/925/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.