Skip to content

Improve the polygon parametrization in the edb layout module. #960

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

Merged
merged 1 commit into from
Mar 16, 2022

Conversation

MaxJPRey
Copy link
Collaborator

_get_edb_value() only take one parameter.
var_server variable is not required and creating a crash if not removed.
The method named calc_slope() in the edb_core\layout.py module could not work properly.
Fix #957

Copy link
Collaborator

@maxcapodi78 maxcapodi78 left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov
Copy link

codecov bot commented Mar 16, 2022

Codecov Report

Merging #960 (384f7fe) into main (b3f7cce) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #960   +/-   ##
=======================================
  Coverage   80.53%   80.53%           
=======================================
  Files         128      128           
  Lines       37855    37855           
=======================================
+ Hits        30485    30486    +1     
+ Misses       7370     7369    -1     

@MaxJPRey MaxJPRey merged commit 3754871 into main Mar 16, 2022
@MaxJPRey MaxJPRey deleted the fix/issue_957 branch March 16, 2022 17:35
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.

_edb_value() expect only one argument.
2 participants