Retry On Error

[erlendp]

Occasionally the render process fails, and it would be desirable to allow the render to immediately try again under certain conditions, such as aerender exiting with a non-zero exit code.

This PR adds the ability for the user to supply a custom onShouldRetry function to trigger a retry as required.

[inlife]

I like the idea! @erlendp do you want to add more work here?

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[sharukh967]

Nice!

[JamesBotterill]

Any movement on this as it would be a very useful thing to have

[JamesBotterill]

Only change I can see that would be needed, is that we limit the number of retries a render can have when erroring out.

Worry could be that we get stuck in a render loop when it's dealing with an issue where an render is erroring out everytime

[JamesBotterill]

@inlife On further review I propose the following changes

1. Right now onShouldRetry is set as event, not as an option flag. This should be set as a flag option "--retry-on-error" (defaulted to false)
2. A second flag "--number-of-retries" should be added as an Number (default to 1) which sets the number of times a render should be retried before erroring out (to avoid infinite loop)

As it looks like OP has abandoned the change, I'll try and find time to do this.

[inlife]

Awesome, will be looking forward. Big thanks for this