Improve and fix problems in geometry generation

JakobTernes commented 4 months ago

This pull request adds functionality to the geometry_generator and fixes several problems:

Additions:

Add version info in geometry file
Add line info in geometry file
Add gui info in readme.md

Fixes:

Wires did not line up at some supertiles -> This was fixed by introducing a WireConstraints object to keep track of wires at the border of a tile and making changes to the geometry generator to take these constraints into account. However, this should be further tested with larger supertiles.
Wires not ordered correctly at termination tiles -> This was resolved by changing the order of the ordering and expanding steps in the geometry generation
FASM not displayed correctly in FABulator -> Partially due to the previous problems and additionally to the IO of ports of bels being inverted. Was resolved by inverting.

I will publish FABulator ( https://github.com/FPGA-Research-Manchester/FABulator ) in the next days. It is currently in its alpha stage and some instabilities and bugs are to be expected.

Btw, i hope this pull request doesn't cause chaos considering a lot is happening here right now...

mole99 commented 4 months ago

Exciting work! Yeah, there's a lot happening right now :D

Since you touched quite some lines, it probably makes sense to merge this before #163 and reapply the formatter on top of it. What do you think @KelvinChung2000?

KelvinChung2000 commented 4 months ago

I will review this first, then with FABulator and then do #163?

Dirk would like to do a 2.0 release, but I am unsure of the exact plan... We will need to clear up the image at some point; we at least need to know what should be included and what not. Otherwise, the release will be a mess.

JakobTernes commented 4 months ago

FABulator is now public.

FPGA-Research-Manchester / FABulous

Improve and fix problems in geometry generation #164