-
Notifications
You must be signed in to change notification settings - Fork 198
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
New sort ports #2788
New sort ports #2788
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We've reviewed this pull request using the Sourcery rules engine. If you would also like our AI-powered code review then let us know.
|
||
|
||
def test_route_bundle_obstacle( | ||
data_regression: DataRegressionFixture, check: bool = True |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
issue (code-quality): We've found these issues:
- Move assignment closer to its usage within a block (
move-assign-in-block
) - Convert for loop into dictionary comprehension (
dict-comprehension
) - Move assignments closer to their usage (
move-assign
)
@@ -736,14 +736,14 @@ def plot_netlist( | |||
) | |||
return G | |||
|
|||
def over_under(self, layer: LayerSpec, distance: float = 1.0) -> None: | |||
def over_under(self, layer: LayerSpec, distance: int = 1) -> None: | |||
"""Flattens and performs over-under on a layer in the Component. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you really need to flatten? I think this makes it strictly bigger, so theoretically it shouldn't be necessary to flatten. But maybe a flag? Otherwise, sure flattening works.
use sort ports from kfactory