Aleksei Fedotov
Aleksei Fedotov
I believe `get_range_split_object` entity becomes unnecessary as well as `explicit`-ness of type conversion operator for `proportional_split` class. Could you please update the patch accordingly?
> @aleksei-fedotov, it is not true. This helper is still required because the implicit conversion from `proportional_split` to `split` is prohibited because of `explicit` operator. The helper is to make...
Hi @Jauler , thanks for the PR! Usually, [composite_node](https://www.threadingbuildingblocks.org/docs/help/hh_goto.htm?index.htm#reference/flow_graph/composite_node_cls.html) is used for combining multiple nodes together to form a subgraph. Could you please describe your preference against composite_node?
The patch looks good to me. Please update the PR description to accurately reflect what this change is about. Better to write by answering the questions what and why.
Priorities specified in the nodes of the graph have nothing to do with the arena priorities, nor do they have with the prioritized nodes from the other graphs. In other...
> > > @aleksei-fedotov @anton-potapov @alexey-katranov, could you please review this patch? Yep. Working on it!
The only way is to count the number of B nodes and their concurrency limitation across all instantiated graphs and make a decision on the concurrency limitation of the new...
> Is it strictly necessary to call wait_for_all() for processing to continue, or is it sufficient to just call it before shutdown? Well, `wait_for_all()` is strictly necessary to call in...
The issue should be fixed by #1777.
> > I believe we need to continue measurements until one of the following conditions are met: > > > > 1. Relative error is within the acceptable limit >...