[Libre-soc-bugs] [Bug 713] PartitionedSignal enhancement to add partition-context-aware lengths

[bugzilla-daemon at libre-soc.org]

https://bugs.libre-soc.org/show_bug.cgi?id=713

--- Comment #81 from Luke Kenneth Casson Leighton <lkcl at lkcl.net> ---
cpart_wid = -min(-lane_shapes[i] // c for i, c in part_counts.items())

that's definitely dividing the lane_shape _total_ by the part count.

it should probably just be

cpart_wid = -min(-lane_shapes[i] for i, c in part_counts.items())

and the input, lane_shapes, should be

0: 5
1: 6
2: 6
3: 6

you definitely gave:

0: 5
1: 6
2: 12
3: 24

check comment #20

for i in range(4):
    l = {0: signed(5), 1: signed(6), 2: signed(12), 3: signed(24)}
    pprint((i, layout(i, part_counts, l)))

-- 
You are receiving this mail because:
You are on the CC list for the bug.