Skip to content

Replace deriv() with explicit dI/dt expressions in V_A..V_E - #24

Merged
sadda merged 1 commit into
mainfrom
15-fix-ddtderiv
Sep 3, 2026
Merged

sadda merged 1 commit into
mainfrom
15-fix-ddtderiv

Conversation

@haitao-ZCU

Copy link
Copy Markdown
Collaborator

Replaces deriv(InputCurrent(...)) in V_A..V_E with explicit dI/dt expressions,

@haitao-ZCU haitao-ZCU linked an issue Sep 3, 2026 that may be closed by this pull request
@sadda
sadda merged commit decbab6 into main Sep 3, 2026
6 checks passed
@smidl

smidl commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Looks correct, so I would merge it.

Three points to check:

  1. Time vs time — new output vars use Time (L109-113), excitation uses time (L215-219).
    Which one is correct? Perhaps both, but potential problem for future. I would prefer to unify it. Check if any is prefered in the documentation

  2. Lew: "0" (L79) makes the new terms numerically inactive — this shows they parse, not that they're right. One solve at Lew=2.4e-3 comparing V_d1 to the analytic value would close it.

However, I think that many of these calcualtion do not belong to ANSYS in teh first place. I will post a separate PR whan this is merged.

@haitao-ZCU

Copy link
Copy Markdown
Collaborator Author

Time vs time is intentional, not an oversight. AEDT needs Time in output variable expressions and time in the excitation source, they are not interchangeable in our testing. I'll add the details to the variable investigation in issue #16 (added as comments first).

@haitao-ZCU
haitao-ZCU deleted the 15-fix-ddtderiv branch September 9, 2026 11:19
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

Successfully merging this pull request may close these issues.

Fix ddt/deriv

3 participants