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

[Feature] Filament center tap on tubes #792

Closed
bogger33 opened this issue May 13, 2024 · 5 comments · Fixed by #798
Closed

[Feature] Filament center tap on tubes #792

bogger33 opened this issue May 13, 2024 · 5 comments · Fixed by #798
Assignees

Comments

@bogger33
Copy link

bogger33 commented May 13, 2024

Is your feature request related to a problem? Please describe.
I wasn't sure whether to file this as a feature request or component request, but some tubes like e.g. the very popular 12AX7 (ECC83) have a center tapped filament, so it can be used with multiple different voltages (either in series or parallel). It would be useful to allow making a connection to the center point of the tube filament in circuitikz to allow for pinout descriptions as well as just more unambiguous circuit diagrams. (It's worth noting that the 12ax7 is a double triode so it might be represented by two triode symbols, but filament splits also exist on tubes which are just single pentodes, etc)

Describe the solution you'd like
An option for connecting to the center point of the filament. I apologize if this already exists and I missed it, but I could not find it in the manual.

Additional context

Center taps are usually represented in diagrams in one of two ways.

Either as two filaments (i.e. two triangles/half-circles each representing one half of the filament):
image

Or just a tap in the center of one filament:
image

It seems to me that the former is more common, but neither is unusual in my estimation, and the latter would probably be simpler to add.

@Rmano
Copy link
Collaborator

Rmano commented May 13, 2024

Hm. In the second example it could be just an anchor, probably (although that will probably need one more \savedanchor, given that the position will depend on the parameters). I'll have a look at it (a pity I just released 😉)

@Rmano Rmano self-assigned this May 13, 2024
@bogger33
Copy link
Author

Sorry about this, I only noticed this just now! However I can work around this so there is no hurry from my side at least

@Rmano
Copy link
Collaborator

Rmano commented May 13, 2024

No problem. Wednesday is a holiday here; maybe I can look at it.

@Rmano
Copy link
Collaborator

Rmano commented May 18, 2024

Hi @bogger33 : what do you think of #798?

@bogger33
Copy link
Author

it looks good to me!

@Rmano Rmano linked a pull request May 19, 2024 that will close this issue
Rmano added a commit that referenced this issue May 19, 2024
Add a center tap for tube's filament

Closes #792
Rmano added a commit to Rmano/circuitikz that referenced this issue May 25, 2024
Several new components, and a bug fix for a nasty long-standing bug about switching diode types.

- Added a Relais-Shape (contributed by [Jakob "DraUX" on GitHub](circuitikz#795)
- Added a center tap anchor for tube's filament (suggested by [user bogger33 on GitHub](circuitikz#792))
- Added neon lamps (two versions, suggested by [user bogger33 on GitHub](circuitikz#793))
- Added a configurable spark gap (suggested by [user bogger33 on GitHub](circuitikz#800))
- Fix a long-standing problem when [(locally) switching diode type](circuitikz#794)
Rmano added a commit to Rmano/circuitikz that referenced this issue May 25, 2024
Several new components, and a bug fix for a nasty long-standing bug about switching diode types.

- Added a Relais-Shape (contributed by [Jakob "DraUX" on GitHub](circuitikz#795)
- Added a center tap anchor for tube's filament (suggested by [user bogger33 on GitHub](circuitikz#792))
- Added neon lamps (two versions, suggested by [user bogger33 on GitHub](circuitikz#793))
- Added a configurable spark gap (suggested by [user bogger33 on GitHub](circuitikz#800))
- Fix a long-standing problem when [(locally) switching diode type](circuitikz#794)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants