sematic-ai / sematic

An open-source ML pipeline development platform
Other
973 stars 59 forks source link

Fix edge-case in timeout comparisons #1096

Closed augray closed 12 months ago

augray commented 12 months ago

There was a bug that if two runs timed out at the same exact time (ex: parallel runs launched simultaneously), the comparison logic would fall back to comparing Future objects directly, which don't support the < operator. This switches so that in such cases we fall back to comparing the ids instead of the Future objects.