search

byuccl / bfasst

Tools for FPGA Assurance Flows
Apache License 2.0
12 stars 5 forks source link

Dump only bel information #485

Closed dallinjdahl closed 5 months ago

dallinjdahl commented 5 months ago

Making output of dump script more like bitstream. Previously, the unique identifier was based on the cell, and we output the cell name as a key property. Now by default the dump script outputs only things relating to bels.

dallinjdahl commented 5 months ago

This works, but neurosub isn't equipped to handle it yet. It is a breaking change to the input format. Even updating neurosub, we'd have to blow away our current datasets and regenerate them.

The advantage is that it's now all bels, all the way down. I'm not sure it would change much in our results, but this is the most accurate bel-dump I know of right now.

dallinjdahl commented 5 months ago

supporting pr in neurosub is https://github.com/byuccl/neurosub/pull/21

jgoeders commented 5 months ago

@dallinjdahl I'm going to merge this and update the datasets