Fix bug in work_dir.

[SuX97]

The args.work_dir will never be None, it will only be '' an empty str. When training without specific work_dir, it results in lots of logs and ckpts saved in $MMACTION2_HOME.

[codecov]

Codecov Report

Merging #110 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #110   +/-   ##
=======================================
  Coverage   85.46%   85.46%           
=======================================
  Files          76       76           
  Lines        4136     4136           
  Branches      661      661           
=======================================
  Hits         3535     3535           
  Misses        494      494           
  Partials      107      107           

Flag	Coverage Δ
#unittests	85.46% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 886a6cf...f1f655c. Read the comment docs.

[innerlee]

[SuX97]

But in slurm_train.sh, it will be
. @innerlee

[innerlee]

that is a bug of slurm_train.sh

[dreamerlin]

1. If we modify line69, line22 should also be changed by setting default value.
2. Otherwise, we could also modify the slurm script.

[innerlee]

2

[SuX97]

I vote for change the slurm scripts, just remove the work_dir to PY_ARGS as the user wishes. @dreamerlin

[dreamerlin]

Well, we can try plan 2 to see what will happen.

[innerlee]

Another choice: abandon slurm_train.sh and adopt slurm_train