Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mlx - merge master into mlx #19657

Merged
merged 110 commits into from
May 3, 2024
Merged

Conversation

lkarthee
Copy link
Contributor

@lkarthee lkarthee commented May 2, 2024

No description provided.

james77777778 and others added 30 commits April 15, 2024 10:06
* Add float8 training support

* Add tests for fp8 training

* Add `quantize_and_dequantize` test

* Fix bugs and add float8 correctness tests

* Cleanup

* Address comments and cleanup

* Add docstrings and some minor refactoring

* Add `QuantizedFloat8DTypePolicy`

* Add dtype policy setter

* Fix torch dynamo issue by using `self._dtype_policy`

* Improve test coverage
* Add LoRA to `BaseConv`

* Add tests

* Fix typo

* Fix tests

* Fix tests
* feat(losses): add Tversky loss implementation

* adjusted documentation
…9522)

- added `Layer.metrics` to return all metrics owned by the layer and its sub-layers recursively.
- `Layer.metrics_variables` now returns variables from all metrics recursively, not just the layer and its direct sub-layers.
- `Model.metrics` now returns all metrics recursively, not just the model level metrics.
- `Model.metrics_variables` now returns variables from all metrics recursively, not just the model level metrics.
- added test coverage to test metrics and variables 2 levels deep.

This is consistent with the Keras 2 behavior and how `Model/Layer.variables` and `Model/Layer.weights` work.
…nd Test Coverage for `Preprocessing Layers` (keras-team#19513)

* Add tests for CategoryEncoding class in category_encoding_test.py

* fix

* Fix IndexLookup class initialization and add test cases

* Add test case for IndexLookupLayerTest without vocabulary

* Fix IndexLookup class initialization

* Add normalization test cases

* Add test cases for Hashing class

* Fix value range validation error in RandomBrightness class

* Refactor IndexLookup class initialization and add test cases

* Reffix ndexLookup class initialization and afix est cases
* API Generator for Keras

* API Generator for Keras

* Generates API Gen via api_gen.sh

* Remove recursive import of _tf_keras

* Generate API Files via api_gen.sh
…eras-team#19529)

This works the same way as in Keras 2, whereby the metrics are returned directly from the logs if the set of keys doesn't match the model metrics.
* Use temp dir and abs path

* Use temp dir and abs path

* Update Readme
… float8 training (keras-team#19534)

* Fix gradient accumulation with `overwrite_with_gradient` in float8 training

* Add comments

* Fix annotation
* Add logic to prevent double quantization

* Add detailed info for double quantization error

* Update error msg
lpizzinidev and others added 14 commits April 28, 2024 11:39
* Fix greedy ctc decode

* Remove print

* Fix `tf.nn.ctc_beam_search_decoder`

* Change default `mask_index` to `0`

* Fix losses test

* Update
…#19636)

* Ensure the same rule applies for np arrays in autocasting

* Trigger CI by adding docstring

* Update

* Update docstring
…as-team#19594)

* Fix and test math functions for jax backend

* run /workspaces/keras/shell/format.sh

* refix

* fix

* fix _get_complex_tensor_from_tuple

* fix

* refix

* Fix istft function to handle inputs with less than 2 dimensions

* fix

* Fix ValueError in istft function for inputs with less than 2 dimensions
…9640)

With Torch, `x.shape` returns a `torch.Size`, which is a subclass of `tuple` but can cause different behaviors. In particular `convert_to_tensor` does not work on `torch.Size`.

This fixes keras-team#18900
* Enable cudnn rnns when dropout is set

* Fix
Bumps the github-actions group with 2 updates: [actions/upload-artifact](https://github.com/actions/upload-artifact) and [github/codeql-action](https://github.com/github/codeql-action).


Updates `actions/upload-artifact` from 4.3.1 to 4.3.3
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@5d5d22a...6546280)

Updates `github/codeql-action` from 3.24.9 to 3.25.3
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@1b1aada...d39d31e)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: github-actions
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: github-actions
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps the python group with 2 updates: torch and torchvision.


Updates `torch` from 2.2.1+cu121 to 2.3.0+cu121

Updates `torchvision` from 0.17.1+cu121 to 0.18.0+cu121

---
updated-dependencies:
- dependency-name: torch
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: python
- dependency-name: torchvision
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: python
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@codecov-commenter
Copy link

codecov-commenter commented May 2, 2024

Codecov Report

Attention: Patch coverage is 88.15713% with 205 lines in your changes are missing coverage. Please review.

❗ No coverage uploaded for pull request base (mlx@4c90dfb). Click here to learn what that means.

Files Patch % Lines
keras/src/backend/common/backend_utils.py 59.64% 32 Missing and 14 partials ⚠️
keras/src/backend/tensorflow/numpy.py 86.75% 23 Missing and 15 partials ⚠️
keras/src/backend/numpy/nn.py 94.52% 5 Missing and 7 partials ⚠️
keras/src/saving/saving_lib.py 80.00% 6 Missing and 6 partials ⚠️
...s/src/trainers/data_adapters/py_dataset_adapter.py 72.50% 9 Missing and 2 partials ⚠️
keras/src/ops/numpy.py 87.65% 6 Missing and 4 partials ⚠️
keras/src/backend/jax/nn.py 95.78% 3 Missing and 5 partials ⚠️
keras/src/utils/model_visualization.py 0.00% 8 Missing ⚠️
keras/api/_tf_keras/keras/ops/__init__.py 0.00% 6 Missing ⚠️
keras/src/backend/tensorflow/nn.py 90.47% 2 Missing and 4 partials ⚠️
... and 18 more
Additional details and impacted files
@@          Coverage Diff           @@
##             mlx   #19657   +/-   ##
======================================
  Coverage       ?   75.56%           
======================================
  Files          ?      508           
  Lines          ?    47212           
  Branches       ?     8681           
======================================
  Hits           ?    35675           
  Misses         ?     9795           
  Partials       ?     1742           
Flag Coverage Δ
keras 75.41% <88.15%> (?)
keras-jax 59.78% <50.54%> (?)
keras-numpy 54.33% <46.73%> (?)
keras-tensorflow 61.08% <58.69%> (?)
keras-torch 59.80% <47.94%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@google-ml-butler google-ml-butler bot added kokoro:force-run ready to pull Ready to be merged into the codebase labels May 3, 2024
@fchollet fchollet merged commit 5a3542b into keras-team:mlx May 3, 2024
6 of 8 checks passed
@google-ml-butler google-ml-butler bot removed awaiting review ready to pull Ready to be merged into the codebase kokoro:force-run labels May 3, 2024
@lkarthee lkarthee deleted the merge_upstream_master branch May 3, 2024 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.