Skip to content

Smith charts need real and imaginary axis inversion #3544

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

Closed
fdion opened this issue Jan 11, 2022 · 2 comments
Closed

Smith charts need real and imaginary axis inversion #3544

fdion opened this issue Jan 11, 2022 · 2 comments

Comments

@fdion
Copy link
Contributor

fdion commented Jan 11, 2022

Current

Smith charts (https://plotly.com/python/smith-charts/) allows one to plot z (impedance chart) from real and imaginary numbers.

This covers only one of 3 common scenarios (impedance, admittance, immittance).

Proposal

Smith charts usually are drawn for z (impedance chart), y (admittance chart) or both z and y (immittance chart). The admittance is an impedance chart rotated 180 degrees, while immittance is a combination of both charts in one.

With a way to reverse both real and imaginary axis on smith charts or to rotate by 180 degrees, it should be possible to properly make all 3 chart types.

@nicolaskruchten
Copy link
Contributor

This is certainly a good idea, and would need to be implemented in Plotly.js. No one from the Plotly team is likely to work on this in the near future, so the most likely way this gets done is via a community contribution. Smith charts were originally introduced in this PR: plotly/plotly.js#5956

@gvwilson
Copy link
Contributor

gvwilson commented Jul 3, 2024

Hi - we are trying to tidy up the stale issues and PRs in Plotly's public repositories so that we can focus on things that are still important to our community. Since this one has been sitting for several years, I'm going to close it; if it is still a concern, please add a comment letting us know what recent version of our software you've checked it with so that I can reopen it and add it to our backlog. Thanks for your help - @gvwilson

@gvwilson gvwilson closed this as completed Jul 3, 2024
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

No branches or pull requests

3 participants