taskflow / taskflow

A General-purpose Parallel and Heterogeneous Task Programming System

Home Page:https://taskflow.github.io

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

WorkerInterface was removed in master

danielytics opened this issue · comments

The WorkerInterface class was removed from master when the exception handling logic was added, however, (the broken) exception handling wasn't the only purpose for WorkerInterface: I've been using tf::WorkerInterface::scheduler_prologue to set thread names for logging and affinity to pin threads to cores. It looks like I'm not the only person using WorkerInterface.

Please consider bringing it back (although obviously the exception parameter is not required in the epilogue function anymore).

@danielytics, would you mind submitting a PR to get this back? I was planning to introduce a more unified interface, but during the interim, we can recover it.

As long as there's a solution planned, I'm happy enough to keep using the latest version that has it. But I will submit a PR for it over the coming days 👍

That will be great! Thank you for the help.