Skip to content

Conversation

@ydcjeff
Copy link
Contributor

@ydcjeff ydcjeff commented Apr 7, 2021

fix #46

Description:

  • merged output_path and filepath into output_dir
  • output_dir is then modified into output_dir / the time right now
  • checkpoints will be stored in output_dir / checkpoints
  • python logging will be stored in output_dir / training-info.log
  • tensorboard outputs will be stored in output_dir

Tree view

output_dir |– 20210407-000000 |– training-info.log |– checkpoints |– best_model_9.pt |– tensorboard_logs |– ... |– 20210407-010000 |– training-info.log |– checkpoints |– best_model_19.pt |– tensorboard_logs |– ... 
@ydcjeff ydcjeff added this to the v0.1.0 milestone Apr 7, 2021
@ydcjeff ydcjeff requested a review from vfdev-5 April 7, 2021 14:46
@vfdev-5
Copy link
Member

vfdev-5 commented Apr 7, 2021

Looks good @ydcjeff , i would just preprend the time with some discrimative config words, e.g.

- 20210407-000000 + resnet18-20210407-000000

otherwise, in practice, we have to enter each folder to get the actual config.
It can be also practical for TB when we'd like to filter experiments.

@ydcjeff ydcjeff merged commit 8dfab13 into master Apr 7, 2021
@ydcjeff ydcjeff deleted the fix/46 branch April 7, 2021 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

2 participants