Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bugfix: file order #156

Conversation

haesleinhuepf
Copy link
Member

Hey guys,

we just had the issue that the file order is random on some systems (Linux, Windows). Thus, if you program a script processing an image sequence on MacOS it works fine using these templates. However, runnning the same script on Linux resulted in errors because the files were processed in random order. Adding the sort-commands, as shown here, solves the issue.

Best,
Robert

@imagejan
Copy link
Member

Batch processing ideally should not depend on file order ;-)
But I agree it's a helpful addition. What about adding some comments explain why we sort here?

👍

@imagejan
Copy link
Member

Also, would you mind formatting the commit message to not be wrapped in the middle of the sentence, @haesleinhuepf ? See the coding style page.
If you force-push to the same branch, I'll merge the PR.

@imagejan imagejan closed this in b630e14 May 22, 2017
@imagejan
Copy link
Member

I went ahead and modified the commit message in b630e14.
🍻

@ctrueden
Copy link
Member

Thanks, guys.

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.

None yet

3 participants