Skip to content

Commit 405c965

Browse files
authored
Add info to README that not only LR is crucial...
... but also the schedule.
1 parent 3c0d6f7 commit 405c965

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

README.md

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,11 @@ achieving about the same performance.
8181
- **This repository requires at least version 1.4 of TensorFlow.**
8282
- **The TensorFlow code is Python 3 only and won't work in Python 2!**
8383

84-
:boom: :fire: :exclamation: **If you train on a very different dataset, don't forget to tune the learning-rate** :exclamation: :fire: :boom:
84+
:boom: :fire: :exclamation: **If you train on a very different dataset, don't forget to tune the learning-rate and schedule** :exclamation: :fire: :boom:
85+
86+
If the dataset is much larger, or much smaller, you might need to train much longer or much shorter.
87+
Market1501, MARS (in tracklets) and DukeMTMC are all roughly similar in size, hence the same schedule works well for all.
88+
CARS196, for example, is much smaller and thus needs a much shorter schedule.
8589

8690
## Defining a dataset
8791

0 commit comments

Comments
 (0)