A typo in dpsim_classes_simulation.svg

sogno-platform / dpsim

Real-time power system simulator including powerflow, (dynamic) phasors and EMT

https://sogno.energy/dpsim/

Mozilla Public License 2.0

67 stars 49 forks source link

A typo in dpsim_classes_simulation.svg #308

Closed georgii-tishenin closed 3 months ago

georgii-tishenin commented 3 months ago

ODESolver class is twice in the dpsim_classes_simulation.svg, used in Architecture | DPsim.

From the context it should be ODESolver and DAESolver. Shall I fix that?

stv0g commented 3 months ago

That would be great :) Is there maybe another new solver class which we could add instead?

georgii-tishenin commented 3 months ago

Good point. I was actually going to just replace one of ODESolvers by DAESolver in the figure. I assume this was a typo from the start.

Though inheritance diagram in Solver class reference has more classes on the same level. The additional ones are DiakopticsSolver, ODEIntSolver and PFSolver.