Add detector_eval_interval to train_from_coco - #3436
Conversation
…t of fixed eval_interval=10 for the detector as in base config
deruyter92
left a comment
There was a problem hiding this comment.
Seems like a good addition to me! This parameter was missing. The implementation is consistent with other parameters.
|
Side note: |
|
@deruyter92 I think I would keep this minimal. The main issue here was the default detector |
Enable to select eval interval for the detector when running adaption_train inside train_from_coco. Currently, there is an eval_interval param which only applies to the pose estimation head. I patched it by adding a new parameter which allows to change eval interval for the detector too. Alternatively, you could standardize eval_interval for both, but I followed the current approach of "detector_" prefix params.