Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

strip-to-slot: center pins around y=0 #323

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mustafacc
Copy link
Member

Slight changes to geometry to make the cell symmetric around y. makes things a lot easier for connect_cell transformations...

Before:
image

After:
image

Slight changes to geometry to make the cell symmetric around y. makes
things a lot easier for connect_cell transformations...
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.30%. Comparing base (066b480) to head (516240e).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #323   +/-   ##
=======================================
  Coverage   75.30%   75.30%           
=======================================
  Files         101      101           
  Lines       12174    12174           
=======================================
  Hits         9168     9168           
  Misses       3006     3006           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@lukasc-ubc
Copy link
Member

Any simulations on the insertion loss? How do the two cells compare?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants