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

Create distribution_lib for TF backend #890

Closed hubingallin closed 10 months ago

hubingallin commented 10 months ago
* Add Mesh and Layout helper functions
codecov[bot] commented 10 months ago

Codecov Report

Patch coverage: 40.00% and project coverage change: -16.09% :warning:

Comparison is base (e8db3b6) 76.56% compared to head (88d3fa2) 60.47%. Report is 72 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #890 +/- ## =========================================== - Coverage 76.56% 60.47% -16.09% =========================================== Files 329 320 -9 Lines 31422 28893 -2529 Branches 6113 5531 -582 =========================================== - Hits 24057 17473 -6584 - Misses 5786 10078 +4292 + Partials 1579 1342 -237 ``` | [Flag](https://app.codecov.io/gh/keras-team/keras-core/pull/890/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/890/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team) | `60.47% <40.00%> (-16.00%)` | :arrow_down: | | [keras_core-numpy](https://app.codecov.io/gh/keras-team/keras-core/pull/890/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team) | `60.47% <40.00%> (?)` | | 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/890?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/\_\_init\_\_.py](https://app.codecov.io/gh/keras-team/keras-core/pull/890?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team#diff-a2VyYXNfY29yZS9iYWNrZW5kL19faW5pdF9fLnB5) | `64.10% <ø> (-30.90%)` | :arrow_down: | | [keras\_core/backend/tensorflow/distribution\_lib.py](https://app.codecov.io/gh/keras-team/keras-core/pull/890?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team#diff-a2VyYXNfY29yZS9iYWNrZW5kL3RlbnNvcmZsb3cvZGlzdHJpYnV0aW9uX2xpYi5weQ==) | `35.71% <35.71%> (ø)` | | | [keras\_core/backend/tensorflow/\_\_init\_\_.py](https://app.codecov.io/gh/keras-team/keras-core/pull/890?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=keras-team#diff-a2VyYXNfY29yZS9iYWNrZW5kL3RlbnNvcmZsb3cvX19pbml0X18ucHk=) | `100.00% <100.00%> (ø)` | | ... and [201 files with indirect coverage changes](https://app.codecov.io/gh/keras-team/keras-core/pull/890/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.

qlzh727 commented 10 months ago

Please also fix the code format via keras_core/shell/{lint|format}.sh

qlzh727 commented 10 months ago

The unit tests are failing. Please check.

fchollet commented 10 months ago

Keras Core is becoming Keras 3, and we're switching development to the main repository! Please reopen this PR in the keras-team/keras repository. Unfortunately we aren't able to automatically transfer PRs (but we have transferred all issues).