CQuad does not order actions based on error estimates

[jbweston]

(original issue on GitLab)

opened by Joseph Weston (@jbweston) at 2017-10-27T14:02:20.393Z

The current implementation (IIRC) is meant to replicate the behaviour of Christoph's code in the case of sequential execution.

Going forward, we should improve this. Currently, if there are any intervals in priority_split, then they are split first. However, priority_split is itself not an ordered collection, so the intervals inside it are ordered in an arbitrary way with respect to the expected loss improvement when splitting them.

[jbweston]

originally posted by Anton Akhmerov (@anton-akhmerov) at 2018-03-10T21:02:12.166Z on GitLab

Depends on gitlab:#50 (priority_split is the higher priority action based on the evaluation whether the error estimate is reliable).