google-research / google-research/circuit_training

Grouping no longer supports large protobuf netlist

Open
#41 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
1.7k
Forks
273
PR merge metrics
No merged PRs in 30d

Description

Hi,
After this [commit](https://github.com/google-research/circuit_training/commit/d5e454e5bcd153a95d320f664af0d1b378aace7b), I see the grouping code no longer supports two smaller split protobuf netlists as input as mentioned [here](https://github.com/google-research/circuit_training/tree/main/circuit_training/grouping#note). So I can not run the latest grouping code for the large protobuf netlist after splitting it into smaller netlist using [split_proto_netlist_main.py](https://github.com/google-research/circuit_training/blob/main/circuit_training/grouping/split_proto_netlist_main.py).

Contributor guide

Open the contributing guide

Research direction

Start by comparing the grouping behavior around commit d5e454e5bcd153a95d320f664af0d1b378aace7b and read the note in circuit_training/grouping. Reproduce the failure using split outputs from split_proto_netlist_main.py; the fix is complete when the latest grouping code accepts the two smaller protobuf netlists described there.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
tooling
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.