Closed vict0rsch closed 4 years ago
Looks like there is a conflict that should be resolved before merging? it's just the name of the branch,I think...
@sashavor you're right there was a conflict because Tianyu commented out a bad import and I removed it so it was conflicting but I just deleted it in the end
Adding a cool feature: Hydra to map command-line args to configuration:
now
python train.py data.loaders.num_workers=10 gen.encoder.n_res=128
will do exactly what you'd expect it to dosee https://hydra.cc (v0.11)
I'll explain how it works when I'm sure it's all good