Ensemble.from_optimization_{history,endpoints} assumes that OptimizerResult.id is always convertible to int

[dweindl]

pypesto.ensemble.ensemble.Ensemble.from_optimization_{history,endpoints} assumes that OptimizerResult.id is always convertible to int

Here:

pyPESTO/pypesto/ensemble/ensemble.py

Line 672 in 82b7d5d

vector_tags.append((int(start["id"]), -1))

and here:

pyPESTO/pypesto/ensemble/ensemble.py

Line 803 in 82b7d5d

(int(result.optimize_result.list[start]["id"]), ind)

Since the attribute is called id (and not index or anything the like), I don't think this is valid.

Those values are used to populate Ensemble.vector_tags: Sequence[tuple[int, int]] which I don't see being used anywhere. Therefore, I'd suggest to just use the id string without attempting conversion to int and changing the type of Ensemble.vector_tags to Sequence[Any].