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.

Sign up for GitHub

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

Jump to bottom

Keras 3 compatibility updates #2165

Closed
wants to merge 45 commits into from
Closed

Keras 3 compatibility updates #2165

wants to merge 45 commits into from

Conversation

divyashreepathihalli
Copy link
Collaborator

@divyashreepathihalli divyashreepathihalli commented Nov 18, 2023 •
edited
Loading

Uses tf.random.Generator instead of keras.backend.RandomGenerator
removed legacy_make_seed

Tests disabled - #2167

  • all 3d preprocessing tests - has legacy baserandomlayer
  • rand_augment_test - need to investigate
  • random_shear_test - need to investigate
  • repeated_augmentation_test - need to investigate
  • resizing_test - has ragged tensor tests and is failing on jax and torch

divyashreepathihalli and others added 27 commits November 18, 2023 04:38
@divyashreepathihalli
Keras3 tests (keras-team#2152)
a9b6c72 
* Update requirements for Keras 3

* Update Requirements

* Update Requirements

* Add randint for keras3 compatibility
@divyashreepathihalli
remove force generator
76c533d
@divyashreepathihalli
fix few tests
c020075
@divyashreepathihalli
fix tests
d67bd7d
@divyashreepathihalli
fix more tests
d6e7eb9
@divyashreepathihalli
Merge branch 'keras-team:master' into tf-generator-changes
121c459
@divyashreepathihalli
Update actions.yml
b6b1117
@divyashreepathihalli @sampathweb
Tf generator changes (keras-team#2162)
f03652f 
* Keras3 tests (keras-team#2152)

* Update requirements for Keras 3

* Update Requirements

* Update Requirements

* Add randint for keras3 compatibility

* remove force generator

* fix few tests

* fix tests

* fix more tests

* Revert "Replace `RandomGenerator` with `SeedGenerator` (keras-team#2150)" (keras-team#2161)

This reverts commit 365a675.

* Update actions.yml

---------

Co-authored-by: Ramesh Sampath <[email protected]>
@divyashreepathihalli
updates to rescaling
f29c781
@divyashreepathihalli
Merge branch 'random-gen-keras-Divya' into tf-generator-changes
fb6c471
@divyashreepathihalli
fix import
29f35eb
@divyashreepathihalli
code reformat
71a415e
@divyashreepathihalli
namescope
bc4341e
@divyashreepathihalli
update name_scope
dca3664
@divyashreepathihalli
correct ops.random
75e10dd
@divyashreepathihalli
update base aug layer
88877f0
@divyashreepathihalli
code reformat
acf3a64
@divyashreepathihalli
skip rand_augment test
c7425c6
@divyashreepathihalli
disable random shear test
78e17e1
@divyashreepathihalli
skip test
dafc9a7
@divyashreepathihalli
update import
d3561ba
@divyashreepathihalli
fix format and base layer
cb6e3f0
@divyashreepathihalli
ads smart resize
3a682bc
@divyashreepathihalli
undo last commit
53b9cad
@divyashreepathihalli
smart_resize update
8ef4d9f
@divyashreepathihalli
update imports
6fd6e40
@divyashreepathihalli
correct shameful spelling mistake
7ab306a
@divyashreepathihalli
Copy link
Collaborator Author

/gcbrun

@sampathweb
Copy link
Collaborator

Replaced by keras-3-updates PR

@sampathweb sampathweb closed this Nov 20, 2023
@divyashreepathihalli divyashreepathihalli deleted the tf-generator-changes branch November 20, 2023 22:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sampathweb sampathweb sampathweb left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@divyashreepathihalli @sampathweb