Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
XCI9 opened this issue Dec 6, 2023 · 3 comments
Open

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

XCI9 opened this issue Dec 6, 2023 · 3 comments

Comments

@XCI9
Copy link

XCI9 commented Dec 6, 2023

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.

@XCI9 XCI9 changed the title The output seems to can't represent some output The output seems to can't represent some situation Dec 6, 2023
@liangrj2014
Copy link
Owner

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
Copy link

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
Copy link
Owner

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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants