EMT Enhanced Documentation#451
Conversation
9f42d0c to
a94de20
Compare
|
@pelesh, I made this PR draft so you can see the range of complexity for the models and their needs. An open-source ULM ( Discuss later. |
53ed821 to
56330ec
Compare
662f657 to
a55ce6a
Compare
pelesh
left a comment
There was a problem hiding this comment.
Looks good. Documentation is consistent and all key parameters and variables are well defined. Some minor formatting issues need to be addressed.
We can merge as soon as @abirchfield approves.
| Bus <Bus/README> | ||
| Component <Component/README> | ||
| Operators <Operators/README> |
There was a problem hiding this comment.
I would separate buses/nodes from other components.
| -------- | ------ | ---------- | ------- | ||
| [`VectorFit`](../../../Operators/Rational/VectorFit/README.md) characteristic admittance $\mathbf{y}_c$ | $\mathbf{v}_{1}\in\mathbb{R}^N$ | `Yc` | $\mathbf{i}^{\mathrm{sh}}_{1}\in\mathbb{R}^K$ | ||
| [`VectorFit`](../../../Operators/Rational/VectorFit/README.md) characteristic admittance $\mathbf{y}_c$ | $\mathbf{v}_{2}\in\mathbb{R}^N$ | `Yc` | $\mathbf{i}^{\mathrm{sh}}_{2}\in\mathbb{R}^K$ | ||
| [`Propagation`](../../../Operators/Shift/Propagation/README.md) $\mathbf{h}$ | $2\mathbf{i}^{\mathrm{sh}}_{2}-\mathbf{i}^{\mathrm{inc}}_{2}$ | `H` | $\mathbf{i}^{\mathrm{inc}}_{1}\in\mathbb{R}^K$ |
There was a problem hiding this comment.
Expressions for H inputs do not render correctly in Github.
|
|
||
| Symbol | Units | JSON | Description | Note | ||
| ---------------- | ------------ | ---- | ---------------------------------- | ---- | ||
| $\mathbf{Z}'$ | [$\Omega$/m] | `Zp` | Series impedance per unit length | $\mathbf{Z}'\in\mathbb{C}^{N \times N}$ |
There was a problem hiding this comment.
Symbol for ohm does not render correctly in GitHub.
| $R_a$ | [$\Omega$] | `Ra` | Load resistance, phase a | | ||
| $R_b$ | [$\Omega$] | `Rb` | Load resistance, phase b | | ||
| $R_c$ | [$\Omega$] | `Rc` | Load resistance, phase c | |
There was a problem hiding this comment.
Adding space after angle bracket helps symbol for ohm render correctly in GitHub (you do get an ugly space though):
| $R_a$ | [$\Omega$] | `Ra` | Load resistance, phase a | | |
| $R_b$ | [$\Omega$] | `Rb` | Load resistance, phase b | | |
| $R_c$ | [$\Omega$] | `Rc` | Load resistance, phase c | | |
| $R_a$ | [ $\Omega$] | `Ra` | Load resistance, phase a | | |
| $R_b$ | [ $\Omega$] | `Rb` | Load resistance, phase b | | |
| $R_c$ | [ $\Omega$] | `Rc` | Load resistance, phase c | |
| \mathbf{R} &= \operatorname{diag}(R_a, R_b, R_c) \\ | ||
| \mathbf{L} &= \operatorname{diag}(L_a, L_b, L_c) |
There was a problem hiding this comment.
Macro \operatorname is not recognized. Try \mathrm:
| \mathbf{R} &= \operatorname{diag}(R_a, R_b, R_c) \\ | |
| \mathbf{L} &= \operatorname{diag}(L_a, L_b, L_c) | |
| \mathbf{R} &= \mathrm{diag}(R_a, R_b, R_c) \\ | |
| \mathbf{L} &= \mathrm{diag}(L_a, L_b, L_c) |
| \mathbf{F}_{\mathrm{in}}(s) &\approx | ||
| \widehat{\mathbf{H}}^{\mathrm{mps}}(s)\mathbf{T}_v(s)^{\mathsf H} \\ | ||
| \boldsymbol{\Delta}_{\tau}(s) &= | ||
| \operatorname{diag}\left(e^{-s\tau_1},\ldots,e^{-s\tau_M}\right) \\ |
There was a problem hiding this comment.
| \operatorname{diag}\left(e^{-s\tau_1},\ldots,e^{-s\tau_M}\right) \\ | |
| \mathrm{diag}\left(e^{-s\tau_1},\ldots,e^{-s\tau_M}\right) \\ |
See earlier comment about macros.
abirchfield
left a comment
There was a problem hiding this comment.
Changes discussed in person, including
(1) Generic N-phase representation, with comments explaining 3-phase will be implemented first
(2) i-dot notation replaced with either i-prime or di/dt
(3) Describing a non-sub-module version of the lumped line model to accelerate implementation of first EMT cases.
(4) Function instead of convolution notation for submodule processes
(5) For submodules, match up the orders and explain any constraints on the parameters
(6) The delay model is for scalars, but used as vector in the propagation model
(7) Delay model needs to note that the second equation is repeated from 1 to n-1.
a55ce6a to
ac01b3b
Compare
7fa7ee3 to
a7f1f84
Compare
Description
Improves documentation for EMT line models and adds documentation for other models needed to implement the most capable version of the Universal Line Model (ULM).
Proposed changes
Linedocumentation forLineLumpedandLineDistributed, replacing the oldBranchline docs (existing model documentation functionally unchanged)LineLumpedwhich is identical to the previous formulation but more general and future compatibleOperatorsgroup for the reusable blocks the models build onRational(VectorFit,StateSpace) for empirical f.d. transmission line modelsShift(Delay,Propagation) for distributed parameter modelingComponent(Bus,Line,Load,Source) andOperatorsChecklist
-Wall -Wpedantic -Wconversion -Wextra.Further comments
All the EMT models are now formulated as ODEs