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

Document weight management for KerasCV #71

Closed
LukeWood opened this issue Jan 31, 2022 · 5 comments
Closed

Document weight management for KerasCV #71

LukeWood opened this issue Jan 31, 2022 · 5 comments
Assignees
@LukeWood
Labels
operational

Comments

@LukeWood
Copy link
Contributor

This includes:

  • default values
  • code locations
  • GCS upload
  • how to reproduce weights
  • where to store reproduction scripts
@LukeWood LukeWood self-assigned this Jan 31, 2022
@bhack
Copy link
Contributor

bhack commented Jan 31, 2022 •
edited
Loading

Some of this points are really important as they are potentially going to expand over the limited reporducbility of the train process. A topic that wasn't handled in the recent PR review at:
keras-team/keras#15447

@bhack
Copy link
Contributor

bhack commented Jan 31, 2022 •
edited
Loading

Just an extra point, I hope that we could evaluate to have some free CI resource for training job resoruces community integrated with Github Action.
But I know that it could be partially connected to this:
tensorflow/build#61 (comment)

@bhack bhack mentioned this issue Feb 1, 2022
Closed
@bhack
Copy link
Contributor

bhack commented Apr 11, 2022

Why we are closing this?

@LukeWood
Copy link
Contributor Author

Why we are closing this?

Project direction has continued to evolve, and while this is still something we will do, this bug is not actionable at this time.

@bhack
Copy link
Contributor

bhack commented Apr 11, 2022

Ok please put a short sentence when you are closing a ticket if it is not super trivial to understand why we are closing it without participating to the internal meetings. So that we could keep the "external" community on the same page. Thanks

freedomtan pushed a commit to freedomtan/keras-cv that referenced this issue Jul 20, 2023
@AakashKumarNain
Add more merging layers (keras-team#71)
e8fdf09 
* add base merge layer

* format docstrings

* add  layer

* add test cases for  layer

* Add import for  layer

* fix build function

* add dynamic and static tests

* fix pytest import

* fix pytest decorator

* remove batch size from dynamic shape test

* fix keras reference

* refactor test class

* fix tf tests, and linting issues

* add subtract layer

* add tests for subtract layer

* fix linting issues

* add average layer

* add maximum layer

* dd minimum layer

* add multiply layer

* add tests for average, minimum, maximum, and multiply layers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@LukeWood LukeWood

Labels
operational
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bhack @LukeWood