close all workers when process is done

[jonSeetrue]

Fix the memory leak issue.
close all workers when done processing.

[jonSeetrue]

Hi @banesullivan
This pr fixes the memory issues of the open workers, could you review it, please?

[banesullivan]

Hi @jonSeetrue, I'm not an active developer or maintainer of this software nor am I familiar with the exact issue referred to here, so I don't think I'm the right person to review this.

Though, as a member of the user community, I think I can speak for everyone in saying thank you for this contribution and likely impactful fix!

[thewtex]

@jonSeetrue yes, thank you for the contribution!

Can you please call terminateWorkers() from a cleanup() function on the viewer object? For performance, we want to re-use the workers on repeated calls.

[jonSeetrue]

@thewtex do you mean to collect all the workers into an array on the viewer object and when loading a new image call terminateWorkers()?

[thewtex]

@jonSeetrue a cleanup() function on the viewer that calls terminateWorkers() on the worker pools when called. Loading a new image will not need a call. I am not sure if an array on the viewer object is needed.

[jonSeetrue]

@thewtex the worker-pool is not exported, so should the cleanup() function be part of each worker-pool implementation?

[thewtex]

@jonSeetrue yes, that could work 👍