EscVM / OIDv4_ToolKit

Download and visualize single or multiple classes from the huge Open Images v4 dataset
GNU General Public License v3.0
800 stars 633 forks source link

Corrected an indentation error #32

Closed monocongo closed 5 years ago

monocongo commented 5 years ago

Fixes a mis-indentation bug which was causing a missing n_threads CLI argument to result in a TypeError, as described in issue #31

keldrom commented 5 years ago

@monocongo can you change only the line that you actually have modified? because like you have done I can't really understand where was the problem

monocongo commented 5 years ago

Sorry, it seems that the tabs were replaced by four spaces (a good idea anyway, but not the point of this fix).

See this comment in issue #31 for an explanation of what this PR fixes. When this indentation is fixed then the code works as advertised, otherwise, you will have errors like what is described in issue #31.

d-j-kendall commented 5 years ago

Oh darn, just opened another issue on this.

keldrom commented 5 years ago

Solved, thx