Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

fix bugs in im2rec.py #3958

Merged
merged 2 commits into from
Nov 24, 2016
Merged

fix bugs in im2rec.py #3958

merged 2 commits into from
Nov 24, 2016

Conversation

chaseleecn
Copy link

fout should be closed before doing anything, if it is not "WindowsError: [Error 32]" will occur.
fname should be removed before renaming fname+".tmp" to fname, if it is not "WindowsError: [Error 183]" will occur.
But why here must rewrite the lst file?

PS, My OS is Windows 10.

Chason added 2 commits November 24, 2016 12:44
fout should be closed before doing anything, if it is not "WindowsError: [Error 32]" will occur.
fname should be removed before renaming fname+".tmp" to fname, if it is not "WindowsError: [Error 183]" will occur.
But why here must rewrite the lst file?

PS, My OS is Windows 10.
@mli
Copy link
Contributor

mli commented Nov 24, 2016

if use multithread, the image order in rec file maybe different to the one given in .lst. so we need to generate a new one to guarantee both files have the same order.

@mli mli merged commit 405d36a into apache:master Nov 24, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants