adaptive icon indicating copy to clipboard operation
adaptive copied to clipboard

Published 20 hours ago • verified publisher icon python-adaptive

potentially change back IntegratorLearner.ivals from a set to a SortedSet

Open basnijholt opened this issue 5 years ago • 0 comments

(original issue on GitLab)

opened by Bas Nijholt (@basnijholt) at 2018-01-02T12:30:21.790Z

In d357a64eb2df8bc7a99c2ecfdc7cf0bebcb8822c we change IntegratorLearner.ivals from a SortedSet (sorted by err) to a normal set because the SortedSet was not always actually sorted, bugs (4bed59da6ae5356f6d95deb25b53e153a432fb88), and it was easier to implement.

We should benchmark the IntegratorLearner and see whether it is worth it to change the set back to a SortedSet.

basnijholt avatar Dec 19 '18 16:12 basnijholt