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

static improvements #1234

Merged
merged 2 commits into from
Nov 7, 2023
Merged

static improvements #1234

merged 2 commits into from
Nov 7, 2023

Conversation

Ravenslofty
Copy link
Collaborator

Two different patches here:

  • from reading the elfPlace paper, the density multiplier and the potential energy should both be vectors; it's not yet clear to me how to change the potential energy, but fixing the density multiplier is easy enough.
  • I added an IO group to iCE40 so that static can place I/O pads when a design doesn't have them specified by PCFs.

@Ravenslofty Ravenslofty force-pushed the lofty/static-improvements branch from 1893767 to 35073f4 Compare November 6, 2023 13:07
@Ravenslofty
Copy link
Collaborator Author

On further reading, it seems more like the density penalty is a scalar in ePlace, but a vector in elfPlace. Still, the code still seems to work fine with this change, so hopefully it's okay?

@gatecat gatecat merged commit e3c44dd into master Nov 7, 2023
16 checks passed
@gatecat gatecat deleted the lofty/static-improvements branch November 7, 2023 07:18
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

Successfully merging this pull request may close these issues.

2 participants