Should this not be min?

[Andful]

stream/stream/classes/hardware/architecture/accelerator.py

Line 321 in ae7ed82

transfer_duration = max([ceil(tensor.size / link.bandwidth) for link in links])

[asyms]

No. The links variable represents the links needed to transfer from Core A to core B. If one of these links has a lower bandwidth, we assume the worst-case transfer time across all of them.

Hope that clarifies your concern.

[Andful]

Bandwidth is inversely proportional to duration. You should either take the max of link.bandwidth or the min of duration. Here you are taking max of duration.

[Andful]

Oh nvm, I re-read your comment, now it makes sense. You are considering the worst-case transfer time.