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

Add current transformer #297

Closed
alexpacini opened this issue Oct 14, 2019 · 9 comments
Closed

Add current transformer #297

alexpacini opened this issue Oct 14, 2019 · 9 comments

Comments

@alexpacini
Copy link
Contributor

@alexpacini alexpacini commented Oct 14, 2019

I noticed that the symbol for the current transformer is missing.
The symbol is the same as the traditional transformer but has only an half circle on one side to show that it is only a single turn.

I wonder if it is worth adding the symbol. I contributed in the past and it should be not too complicated to make it from the transformer (just leave half a turn), but unfortunately I don't have lot of time available at the moment. Therefore I am not sure if I can make it in a reasonable time (and do it properly with docs, etc.).
If anyone is willing to do that I will appreciate and will say thanks :)

@Rmano
Copy link
Collaborator

@Rmano Rmano commented Oct 14, 2019

Could you please add an example? I am not sure to understand the description...

@alexpacini
Copy link
Contributor Author

@alexpacini alexpacini commented Oct 14, 2019

Sure. I apologise for the low quality...
The style should be the same as the transformers, but with a single turn.
The turn can have the same size as the turns on the other sides (in the picture it looks bigger).

current_trafo

@Rmano
Copy link
Collaborator

@Rmano Rmano commented Oct 15, 2019

I am thinking to allow styling of the inductors, so that you can have diagrams like these:

image

@alexpacini
Copy link
Contributor Author

@alexpacini alexpacini commented Oct 15, 2019

That is very nice.
Consider though that the current sensing side (the one with a single loop) is often placed inline with the line, therefore without the two "knees" (similarly to my picture).

@Rmano
Copy link
Collaborator

@Rmano Rmano commented Oct 15, 2019

This could be achieved with the quadpoles style=inline (which I prefer anyway):

image

@Rmano
Copy link
Collaborator

@Rmano Rmano commented Oct 15, 2019

Hmm... there is a bug in the core of the inline transformer core --- will check.

@Rmano
Copy link
Collaborator

@Rmano Rmano commented Oct 15, 2019

Hmm... there is a bug in the core of the inline transformer core --- will check.

Will be fixed by #298

@alexpacini
Copy link
Contributor Author

@alexpacini alexpacini commented Oct 15, 2019

Perfect!
Do you also need to set the coil width?

@Rmano
Copy link
Collaborator

@Rmano Rmano commented Oct 16, 2019

Do you also need to set the coil width?

Yes, because the number of coils and the width are independent. But you can easily define a style:

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

2 participants