Skip to content

Conversation

@ssnl
Copy link
Contributor

@ssnl ssnl commented Nov 17, 2017

Fixes several issues so that the training commands in README work:

  1. fix os.execute -> os.system
  2. fix hacks for data_parallel. They weren't working in data parallel because the old lambda function refers to tensors not transferred to proper GPU yet.
  3. fix train/val/test directory checking in COCO image loading. Current code only checks the containing dir of these split sets so it errors in cases when downloading val after train.
  4. fix training config yaml missing size for coco, and thus can't find the correct directory to load data.

@Cadene
Copy link
Owner

Cadene commented Nov 18, 2017

Thanks so much!

@Cadene Cadene merged commit b4751f5 into Cadene:master Nov 18, 2017
@ssnl ssnl deleted the os_system branch November 18, 2017 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants