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

Simple improvements for better parallelization #2

Merged
merged 5 commits into from
Apr 10, 2018

Conversation

sjuxax
Copy link
Contributor

@sjuxax sjuxax commented Apr 10, 2018

Some simple improvements that should make it easier to run this script in parallel. I'm using these to convert directories with hundreds of 360 images taken with my Theta V.

This allows us to execute the script simultaneously within the same
working directory. For example, GNU parallel can be used to run several
instances at once. Before, the instances would stomp on each other,
prematuerly replacing or destroying the contents of the temp files.
This makes it easier to resume processing of large directories.
@khufkens
Copy link
Owner

Nice addition to the script, thanks!

Would you mind replacing the . delimiters to _ ones just to remain consistent? I use . for extensions and _ for file name parts in general as it increases readability across languages.

@sjuxax
Copy link
Contributor Author

sjuxax commented Apr 10, 2018

Sure thing. Done.

@khufkens khufkens merged commit ad90497 into khufkens:master Apr 10, 2018
@khufkens
Copy link
Owner

Merged, thanks again for the addition and the fix. Have fun with the Theta camera!

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