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

QucsAttenuator: Ensure Zout=Zin in QW and Lpad attenuators #880

Merged

Conversation

andresmmera
Copy link
Contributor

Hello:

I have found that when selecting the QW or L-pad attenuators and marking the option to include the S-parameter analysis block, the impedance of the output term was not $Z_0$ if $Z_0 \neq 50 \Omega$

This happened because these attenuator topologies were missing from slotSetText_Zin(). This function calls the calculation function in order to update the power dissipation data whenever the user changes $Z_0$.

Steps to reproduce

  1. Select a L-pad attenuator (or a QW attenuator) and set $Z_0 =75 \Omega$

  2. Check the S-parameter box option
    Selection_034

  3. Paste the schematic in Qucs-S: The impedance of the output port is not $75 \Omega$, but $50 \Omega$
    Selection_036

Solution

Simply add QW and Lpad attenuators to slotSetText_Zin()
Selection_035

It was found that when selecting the QW or L-pad attenuators and marking the option to include the S-parameter analysis block, the impedance of the output term was not Z0 if Z0 != 50

This happened because these attenuator topologies were missing from slotSetText_Zin( double val ). This function calls the calculation function in order to update the power dissipation data whenever the user changes Z0.
@andresmmera andresmmera changed the title QucsAttenuator: Ensure Zout=Zin in QW and Lpad attenuatores QucsAttenuator: Ensure Zout=Zin in QW and Lpad attenuators Aug 1, 2024
@ra3xdh ra3xdh added this to the 24.4.0 milestone Aug 1, 2024
@ra3xdh ra3xdh merged commit cf9c5e1 into ra3xdh:current Aug 1, 2024
7 checks passed
@ra3xdh
Copy link
Owner

ra3xdh commented Aug 1, 2024

Thanks! Merged.

@ra3xdh ra3xdh modified the milestones: 24.4.0, 24.3.1 Sep 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants