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

Adding BinaryDice Loss #1050

Open
wants to merge 64 commits into
base: master
Choose a base branch
from
Open

Adding BinaryDice Loss #1050

wants to merge 64 commits into from

Conversation

DavidLandup0
Copy link
Contributor

@DavidLandup0 DavidLandup0 commented Nov 23, 2022 •
edited by jbischof
Loading

What does this PR do?

Introduced in #296, discussed in #371, separation requested in #968.

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you read the contributor guideline,
    Pull Request section?
  • Was this discussed/approved via a Github issue? Please add a link
    to it if that's the case.
  • Did you write any new necessary tests?
  • If this adds a new model, can you run a few training steps on TPU in Colab to ensure that no XLA incompatible OP are used?

Who can review?

@LukeWood @tanzhenyu

innat and others added 30 commits April 27, 2022 04:27
@innat
init dice loss
277fb8d
@innat
update
bde5fad
@innat
refactor
7e29651
@innat
update check and refactor
1f4f104
@innat
simplified and refactor
65c9a6b
@innat
fix_name_init and update check logic
460643d
@innat
add dice_test
2dd3fe1
@innat
fix pytest
c7dbe61
@innat
add get_config
d3c5bc8
@innat
add axis param to support 2d and 3d input
8c16199
@innat
add loss type params
a5cd22b
@innat
fix typo
de24048
@innat
backend to tf.math
285fa9c
@innat
make serializable
ea03cc3
@innat
fix typo
c4c4d2a
@innat
relocate gather channel func
ab1b8d5
@innat
reform init params and describe
d739c60
@innat
add dice_variants
bb8127a
@innat
update
c9c8008
@innat
remove redundant test
6bd631e
@innat
reorganize to concise
32fb39b
@innat
remove non-use param
be1c14e
@innat
typos
c298def
@DavidLandup0
Training script improvement proposal
ffc1fd2
@DavidLandup0
Added flags to customize warmup and hold percentages
c7daf26
@DavidLandup0
Added TODO for changing custom schedule with the future core keras sc…
681b302 
…hedule
@DavidLandup0
Changed epochs to be controlled by a flag
871f127
@DavidLandup0
Generalized learning_rate_warmup flag to both schedules
8e2ecd4
@DavidLandup0
Added constants for strings and renamed lr warmup
9789251
@DavidLandup0
Hardcoded PI -> np.pi
50c2438
DavidLandup0 and others added 28 commits October 27, 2022 11:04
@DavidLandup0
resolved merge conflict
b3852d7
@DavidLandup0
various typo fixes
1f48010
@DavidLandup0
Merge branch 'keras-team:master' into dice
9438290
@DavidLandup0
skeleton for tests
7b83508
@DavidLandup0
renamed file to conform to other losses
28199ec
@DavidLandup0
renamed back...
8dd7063
@DavidLandup0
tests for dice loss and typo fixes
80d0904
@DavidLandup0
docs touchup
0d54896
@DavidLandup0
Fixed issue ignoring input channel position, added checks for axis ar…
e89a8dd 
…gument
@DavidLandup0
Fixed axis value check
243b15f
@DavidLandup0
added per_image flag
04a34d9
@DavidLandup0
formatting
be2c531
@DavidLandup0
removed unused import
3815fc5
@DavidLandup0
removed unused import
fa69da2
@DavidLandup0
sync:
d9eb0a1
@DavidLandup0
Merge branch 'keras-team:master' into dice
44a09ec
@DavidLandup0
Merge branch 'dice' of https://github.com/DavidLandup0/keras-cv into …
5bbbd4f 
…dice
@DavidLandup0
fixed sparse dice loss
dd45f00
@DavidLandup0
is it fixed? vsauce music
bbb8550
@DavidLandup0
removal of print statement
55f01ef
@DavidLandup0
sync
464cf6c
@DavidLandup0
sync
fc39d3a
@DavidLandup0
fixed tests
c98ad00
@DavidLandup0
.
dd5ea9c
@DavidLandup0
Fixed tests, enhanced docs
e4deca2
@DavidLandup0
separated binary dice
6a96eaf
@DavidLandup0
removed imports
f004f8f
@DavidLandup0
imports
dbf53e5
@innat innat mentioned this pull request Feb 23, 2023
Closed
@innat innat mentioned this pull request May 28, 2023
Open
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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
@DavidLandup0 @innat