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

Filter NULL tiles in synth tile map to vpr coord #1549

Merged
merged 4 commits into from Jun 29, 2020

Conversation

andrewb1999
Copy link
Collaborator

Signed-off-by: Andrew Butt andrewb1999@gmail.com

Filters NULL tile types when mapping synth tiles to vpr coordinates. Fixes #1538

Signed-off-by: Andrew Butt <andrewb1999@gmail.com>
@probot-autolabeler probot-autolabeler bot added lang-python Issue uses (or requires) Python language. type-utils Issues is related to the scripts inside the repo. labels Jun 25, 2020
Signed-off-by: Andrew Butt <andrewb1999@gmail.com>
Signed-off-by: Andrew Butt <andrewb1999@gmail.com>
@litghost
Copy link
Contributor

Does this PR allow you to define synth IO's where you want (excepting #1550)?

@andrewb1999
Copy link
Collaborator Author

Does this PR allow you to define synth IO's where you want (excepting #1550)?

Just double checked, and no, it looks like this version does not fix all issues I was having. Going to try figuring out why next and will update.

@andrewb1999
Copy link
Collaborator Author

Messed up the table join. Fixing now.

Signed-off-by: Andrew Butt <andrewb1999@gmail.com>
@andrewb1999
Copy link
Collaborator Author

@litghost Why did the iCE40 check fail? To me it looks like a random fail. Don't think my changes should affect iCE40 at all.

@litghost
Copy link
Contributor

('Connection broken: OSError("(104, \'ECONNRESET\')")', OSError("(104, 'ECONNRESET')"))

Looks like a temporary network disruption.

Copy link
Contributor

@litghost litghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, will review CI data once it runs again.

@litghost litghost merged commit cb53a58 into f4pga:master Jun 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang-python Issue uses (or requires) Python language. type-utils Issues is related to the scripts inside the repo.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Multiple mapped tiles when creating synth io tiles
3 participants