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.

Sign up for GitHub

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

Jump to bottom

Branch: fix_check_route_tree #321

Closed
4 tasks
litghost opened this issue Nov 7, 2019 · 1 comment
Closed
4 tasks

Branch: fix_check_route_tree #321

litghost opened this issue Nov 7, 2019 · 1 comment

Comments

@litghost
Copy link

litghost commented Nov 7, 2019

Why did we need this? (what does this change enable us to do)

Fixes assertion violation related to C_internal

What did it change?

Updates is_valid_route_tree to account for C_internal

Should it be merged upstream - if not, when can we delete it?

Upstream PR is already created: verilog-to-routing#1042

What is needed to get this merged / deleted?

  • is the implementation work to make suitable for merging / deletion completed?
  • Is there an associated test?
  • is this currently part of the Conda package?
  • is this properly cleaned up in our local repositories?

Tracker / branch / PR & other useful links
@litghost
Copy link
Author

verilog-to-routing#1042 has been merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@litghost