-
Notifications
You must be signed in to change notification settings - Fork 0
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
process image in parallel #99
Conversation
This is one of the first webpage I saw yet I magically dismissed it and proceeds to spend 10 hours on useless solution that lead me back to where I started
non reproducable bug with commit 3f1c0bc, not sure if still here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've made changes where I think is appropriate on another branch, so you can just fast-forward to that. richard/imageProcess5...shangzhel/imageProcess5
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
closes #78.
Please check tsconfig in case I did something I'm not supposed to do.
Do feel free to point out coding style improvements too.
Thanks