search

mrsnu / band

Multi-DNN Inference Engine for Heterogeneous Mobile Processors
Other
23 stars 2 forks source link

[Bug] `Job.expected_latency` is updated by nobody #292

Closed widiba03304 closed 1 year ago

widiba03304 commented 1 year ago

LSF scheduler utilizes job.expected_latency to calculate the slack time but nobody sets it :(

dostos commented 1 year ago

I believe it's the scheduler's responsibility to update the expected_latency (expected latency of the designated Subgraph of Job) as LSF currently does. https://github.com/mrsnu/band/blob/578f0d294ca19cf3b42bb8478e0fc895278be3d7/band/scheduler/least_slack_first_scheduler.cc#L79-L98

widiba03304 commented 1 year ago

@dostos My bad. Sorry for the false alarm!