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

Improve voltage computation for lumped ports #1519

Merged
merged 1 commit into from
Mar 8, 2024

Conversation

dmarek-flex
Copy link
Contributor

@dmarek-flex dmarek-flex commented Mar 6, 2024

The E field is discontinuous at the port boundaries where conductors are present. In this picture the ground plane is at z=0 and the strip is at z=1000. Interpolation is inaccurate there, and even though only the end points are affected, it has a noticeable effect on accuracy.

image

Now, all field points outside the port bounds are ignored and linear extrapolation is used to estimate the E field on the exact port bounds.

@dmarek-flex dmarek-flex self-assigned this Mar 6, 2024
@dmarek-flex dmarek-flex marked this pull request as ready for review March 6, 2024 19:53
Copy link
Collaborator

@tylerflex tylerflex left a comment

Choose a reason for hiding this comment

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

Looks good to me, was going to say to include a changelog item but it probably isn't necessary since the original version of this hasn't been released yet, right?

@dmarek-flex
Copy link
Contributor Author

Looks good to me, was going to say to include a changelog item but it probably isn't necessary since the original version of this hasn't been released yet, right?

Yea I was about to add something to the changelog and saw the lumped ports stuff right above, so I decided not to change anything.

@dmarek-flex dmarek-flex merged commit b59c8e7 into pre/2.7 Mar 8, 2024
13 checks passed
@dmarek-flex dmarek-flex deleted the dmarek/improve_voltage_comp branch March 8, 2024 13:56
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.

None yet

3 participants