liangrj2014 / ISPD24_contest

20 stars 0 forks source link

The output seems to can't represent some situation #8

Open XCI9 opened 11 months ago

XCI9 commented 11 months ago

For the given example in spec:

Net0
{
0 0 1 1 metal1
0 0 1 3 metal2
0 2 4 3 metal3
3 2 4 4 metal2
3 3 4 4 metal1
}

image

According to the solution, it seems that the solution is output all covered Gcell but not all used edges since (0, 2) should have a via but the solution does not cover it.

For the following net, red means wires(same as example in spec, horizontal in metal3 and vertical in metal2), blue means pins, green means vias, note that in (1, 2), I don't want to have a via there since it is congestion there. But since Gcell (1, 2, metal2) and (1, 2, metal3) are adjecent Gcell, it will be viewed as a via there, which is not what I want. image Is there any way to solve this?

If I have misunderstood anything, please let me know. Thanks.

liangrj2014 commented 11 months ago

You are right! We plan to update the output format and the evaluator to accommodate it. To be specific, the output formate will also indicate the location of vias. Here is an example:

Net0

{

x_l y_l z_l x_h y_h z_h

0 0 0 0 0 0

0 0 0 0 0 1 # a via

0 0 1 0 2 1

0 2 1 0 2 2 # a via

...

} where each row (x_l y_l z_l x_h y_h z_h) describes a line/rectangle in the 3D GCell graph, spanning from (x_l, y_l, z_l) to (x_h, y_h, z_h).

We will release the updated output format soon. And our evaluator will also be updated accordingly. Thanks!

cai-zhi-jie commented 11 months ago

a few minor questions:

  1. is the x_h, y_h in the updated output format is included int the solution, as I see, in the current evaluator and contest introduction, x_h and y_h is not included as shows follow: image image but since the example you gave include "0 0 0 0 0 0", so I assume the x_h and y_h will be included in the updated output format?
  2. can the via go through multiple layers in the update output format? for example "0 0 0 0 0 4"
  3. will the update announced in some place for us to catch up in time? (including the release/update of benchmark, the update of evaluate script/metric, the update of input/output format and so on)
liangrj2014 commented 11 months ago

Thanks a lot for your questions!

  1. You are right. The x_h and y_h will be included in the updated output format.
  2. Yes, it can.
  3. We are actively working on these. We expect to release the updated (hopefully final version) of evaluate script/metric and input/output format by the end of this week. And more benchmarks will be released in the next week, including those with up to 50M cells

Really appreciate your questions and patience!