forked from sharpie7/circuitjs1
-
Notifications
You must be signed in to change notification settings - Fork 300
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
[Bug] Diode not dioing #137
Comments
Yout assumption is wrong, the default diode model is not an ideal diode. See also #88. But I agree that combining logic inputs (which draw no current) with diodes leads to confusing behavior. Maybe there should be a model for an ideal diode? |
Greetings,
i see now. Thank you for the clarification and sorry for the
misunderstanding.
Do i delete this issue now? I am sort of new to this kind of thing. Also
thank you for taking time to review this.
Have a nice day/evening.
…-Lipino
On Wed, Jan 8, 2025, 21:04 Fritz Webering ***@***.***> wrote:
Yout assumption is wrong, the default diode model is not an ideal diode.
See also #88 <#88>.
But I agree that combining logic inputs (which draw no current) with
diodes leads to confusing behavior. Maybe there should be a model for an
ideal diode?
—
Reply to this email directly, view it on GitHub
<#137 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A6FJATAAN342HSSTB2OQH4T2JWAEFAVCNFSM6AAAAABUZ7O336VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNZYGUZTSMBVGI>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I think the picture is selfexplenatory:
WhatInTheDiode.txt
it just goes trough. i know diodes IRL can let some tiny ammount of current trough, but I assume this default diode is supposed to be a theorethically perfect diode
The text was updated successfully, but these errors were encountered: