staudtMarius
staudtMarius
This could be closed after the PSDM is updated. See [PSDM 1077](https://github.com/ie3-institute/PowerSystemDataModel/issues/1077).
I found the issue.
@sebastian-peter To do this we'd have to change classes like `AssetInput` and `UniqueInput` from being abstract. This should be done after #1044.
@sebastian-peter If you want, we could talk about this next week.
I think this could be partially resolved with #999. After this PR an exception is thrown if we cannot read the csv weather or csv coordinate source correctly.
@t-ober Maybe use could use [this](https://github.com/ie3-institute/PowerSystemDataModel/pull/1130) PSDM PR to read in a voltage time series.
> Just as a reminder since I just checked the old issues. Could / should we handle #136 here as well? I think we could just add this in this...
Maybe we should use the cosPhiRated here to calculate the active power.  @danielfeismann, @sebastian-peter
> Yes, I agree We cannot use the cosPhi there. This should be `Q = sqrt(S^2 - P^2)`. I adapted this part and added a comment.
@danielfeismann Would it be possible to also merge #823 before?