staudtMarius

Results 23 comments of staudtMarius

This could be closed after the PSDM is updated. See [PSDM 1077](https://github.com/ie3-institute/PowerSystemDataModel/issues/1077).

@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. ![image](https://github.com/user-attachments/assets/ec4fb5e4-3758-4ade-b028-7f1bdafe1cfc) @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?