UniversalGRAMM / UGRAMM

Universal GRAph Minor Mapper (UGRAMM)
https://universalgramm.github.io/
Other
2 stars 0 forks source link

Add pins to the printName function output #3

Closed HamasWaqar closed 1 month ago

HamasWaqar commented 1 month ago

Hey Omkar,

I do not see what pins are being used for the specific tile as it is not printed in through the printName function. Could you add this feature to the design?

Working on Branch: new_pin_layout_fix

Screenshot 2024-08-15 at 10 18 34 AM Screenshot 2024-08-15 at 10 21 41 AM
HamasWaqar commented 1 month ago

To reproduce this, I just ran the following command ./run_gramm.sh Kernels/Conv_Balance/modified_conv_nounroll_Balance_Tester.dot 8 8 Riken > ./logs/log.txt

ombhilare999 commented 1 month ago

Hi @HamasWaqar ,

This issue should be fixed after the commit 88a0318dd32ac96a409a0ac15008a6c22aa1c8f6

image

FADD_19, FADD_18 should be connected to the input pins of FADD_25.

As you can see in the output:

Let me know if you find any more issues in this branch.

The run command I used: ./run_gramm.sh Kernels/Conv_Balance/conv_nounroll_Balance.dot 8 8 RIKEN_with_pins

ombhilare999 commented 1 month ago

Solve with the PR #5