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

Update gridmask for v1 #287

Merged
merged 4 commits into from
Apr 11, 2022
Merged

Update gridmask for v1 #287

merged 4 commits into from
Apr 11, 2022

Conversation

LukeWood
Copy link
Contributor

@LukeWood LukeWood commented Apr 8, 2022

closes #285 #284 and #283

Image from demo:

Figure_1

@LukeWood LukeWood requested a review from qlzh727 April 8, 2022 20:02
@LukeWood
Copy link
Contributor Author

LukeWood commented Apr 8, 2022

We can't update rotation factor until we introduce factor samplers to keras core. Let's target next release for this (tensorflow 2.10)

@LukeWood
Copy link
Contributor Author

LukeWood commented Apr 8, 2022

FYI @qlzh727

@LukeWood LukeWood merged commit f35bca5 into master Apr 11, 2022
@LukeWood LukeWood deleted the gridmask-fix branch April 14, 2022 18:51
kartik4949 pushed a commit to kartik4949/keras-cv that referenced this pull request Apr 21, 2022
* Update gridmask for v1

* Update gridmask

* Format

* fix serialize
LukeWood added a commit to kartik4949/keras-cv that referenced this pull request Apr 29, 2022
* Update gridmask for v1

* Update gridmask

* Format

* fix serialize
ianstenbit pushed a commit to ianstenbit/keras-cv that referenced this pull request Aug 6, 2022
* Update gridmask for v1

* Update gridmask

* Format

* fix serialize
adhadse pushed a commit to adhadse/keras-cv that referenced this pull request Sep 17, 2022
* Update gridmask for v1

* Update gridmask

* Format

* fix serialize
freedomtan pushed a commit to freedomtan/keras-cv that referenced this pull request Jul 20, 2023
…as-team#287)

* Eager torch benchmark

* small fix

* add a simpler benchmark

* small fix

* refactor the code

* refactor

* Add readme
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GridMask ratio argument should be inverted
1 participant