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

Correct lingering bugs in xc7 route timing import. #742

Closed
wants to merge 1 commit into from

Conversation

litghost
Copy link
Contributor

No description provided.

@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 May 21, 2019
@litghost litghost requested review from mithro and HackerFoo May 21, 2019 14:30
Copy link
Contributor

@mithro mithro left a comment

Choose a reason for hiding this comment

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

LGTM.

@mithro
Copy link
Contributor

mithro commented May 29, 2019

@litghost What is the status of this pull request?

Signed-off-by: Keith Rothman <537074+litghost@users.noreply.github.com>
Signed-off-by: Tim 'mithro' Ansell <me@mith.ro>
@mithro
Copy link
Contributor

mithro commented May 29, 2019

I rebased this onto master to see if it works.

@litghost
Copy link
Contributor Author

This PR requires the changes in SymbiFlow/vtr-verilog-to-routing#57, which have not been merged and incorporated into conda yet.

@mithro
Copy link
Contributor

mithro commented Jul 9, 2019

@litghost I think this can be rebased now?

@litghost
Copy link
Contributor Author

litghost commented Jul 9, 2019

@litghost I think this can be rebased now?

This PR is dead.
Superceeded in: #849

@litghost litghost closed this Jul 9, 2019
@litghost litghost deleted the fixup_route_timing branch September 18, 2019 17:14
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. third-party type-utils Issues is related to the scripts inside the repo.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants