ratt-ru / QuartiCal

CubiCal, but with greater power.
MIT License
7 stars 4 forks source link

Restringify keys in scheduler plugin. #322

Closed JSKenyon closed 5 months ago

JSKenyon commented 5 months ago

Dask has reverted to tuple keys (from string) - see #315. This workaround restringifies the keys in the scheduler plugin to avoid errors. This is a temporary solution, but it should be adequate until I have a chance to rewrite/improve the plugin.

JSKenyon commented 5 months ago

The graph keys which expose this problem come from from_array calls. They are not tuples, but simple strings. In order to sort these values consistently, we need to make all of them tuples, or all of them strings. Inlining the from_array calls also addresses the problem. Stringifying is adequate for now, but this may need to be revisited.