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

adding deeplab_v3_plus_presets #2051

Merged
merged 22 commits into from
Sep 20, 2023
Merged

adding deeplab_v3_plus_presets #2051

merged 22 commits into from
Sep 20, 2023

Conversation

divyashreepathihalli
Copy link
Collaborator

No description provided.

@divyashreepathihalli
Copy link
Collaborator Author

/gcbrun

Copy link
Contributor

@ianstenbit ianstenbit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you Divya! Looks good, just a few little tweaks.

Copy link
Contributor

@ianstenbit ianstenbit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Just a quick question about the preset bit -- @jbischof curious about your take as well. (IMO we should make this from_preset change on the Task class instead of just for DeepLabV3Plus

Copy link
Contributor

@ianstenbit ianstenbit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM -- @jbischof PTAL quickly to verify that you're cool with this change to from_preset

@ianstenbit
Copy link
Contributor

/gcbrun

Copy link
Contributor

@jbischof jbischof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks look amazing!

keras_cv/models/task.py Outdated Show resolved Hide resolved
@ianstenbit
Copy link
Contributor

/gcbrun

2 similar comments
@ianstenbit
Copy link
Contributor

/gcbrun

@ianstenbit
Copy link
Contributor

/gcbrun

keras_cv/models/task.py Outdated Show resolved Hide resolved
@divyashreepathihalli
Copy link
Collaborator Author

/gcbrun

@ianstenbit
Copy link
Contributor

/gcbrun

@ianstenbit
Copy link
Contributor

@divyashreepathihalli test_with_model_preset_forward_pass is failing due to the numerics not matching, which is more or less what I'd expect -- shouldn't it be close to all 1s for the background class and all 0s for the non-background class?

@divyashreepathihalli
Copy link
Collaborator Author

/gcbrun

@divyashreepathihalli
Copy link
Collaborator Author

/gcbrun

@ianstenbit ianstenbit merged commit 05263cf into keras-team:master Sep 20, 2023
9 checks passed
ghost pushed a commit to y-vectorfield/keras-cv that referenced this pull request Nov 16, 2023
* adding deeplab_v3_plus_presets

* update formatting

* fixed error

* reformatted

* fix errors

* added num_classses arg

* add input_shape arg

* update test

* update input_shape

* updated preset

* update test

* move update to from_preset to task.py

* reformatted code

* updated doc string

* Update assert

* Update deeplab_v3_plus_test.py test

* Update task.py

* Update deeplab_v3_plus_test.py

* Update deeplab_v3_plus_test.py
yuvraj-wale pushed a commit to yuvraj-wale/keras-cv that referenced this pull request Feb 8, 2024
* adding deeplab_v3_plus_presets

* update formatting

* fixed error

* reformatted

* fix errors

* added num_classses arg

* add input_shape arg

* update test

* update input_shape

* updated preset

* update test

* move update to from_preset to task.py

* reformatted code

* updated doc string

* Update assert

* Update deeplab_v3_plus_test.py test

* Update task.py

* Update deeplab_v3_plus_test.py

* Update deeplab_v3_plus_test.py
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.

None yet

3 participants